Skip to content

Conversation

infrahub-github-bot-app[bot]
Copy link

Merging develop into infrahub-develop after merging pull request #551.

minitriga and others added 11 commits September 8, 2025 17:12
…o times. (#531)

* feat: add create_diff method and update get_diff_summary to accept name parameter

* feat: enhance get_diff_summary to support time range filtering

* fix: update action field in diff summary to reflect correct status

* feat: add create_diff method and update get_diff_summary for time range support

* feat: add create_diff method and enhance get_diff_summary to support time range filtering

* fix: remove unnecessary blank lines in InfrahubClientSync class

* feat: update create_diff method to validate time range and change return type
…ables (#536)

* Fix branch handling in `_run_transform` and `execute_graphql_query` functions

* Add changelog entry for fixing branch handling in `_run_transform` and `execute_graphql_query` functions

* Remove unused import of `get_branch` from `utils.py`

* Add jinja2 transform and GraphQL query for tags, and implement branch selection test

* Update changelog to specify environment variable usage for branch management in `_run_transform` and `execute_graphql_query` functions.

* Remove unused `get_branch` import and set default branch in `validate_graphql` function
Remove unnecessary assignments before `return` statement
fix improperly escaped special characters in `HFID`
* Replace toml package with tomllib and tomli optionally

* fix for mypy errors

* update config, add pre-commit config

* Make tomli package optional

* Add towncrier housekeeping message

* update python to uppercase p
Copy link

coderabbitai bot commented Sep 17, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot added group/ci Issue related to the CI pipeline type/documentation Improvements or additions to documentation labels Sep 17, 2025
Copy link

cloudflare-workers-and-pages bot commented Sep 17, 2025

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2c16773
Status: ✅  Deploy successful!
Preview URL: https://13776d82.infrahub-sdk-python.pages.dev
Branch Preview URL: https://develop.infrahub-sdk-python.pages.dev

View logs

Copy link

codecov bot commented Sep 17, 2025

Codecov Report

❌ Patch coverage is 73.77778% with 59 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/client.py 35.00% 29 Missing and 10 partials ⚠️
infrahub_sdk/spec/range_expansion.py 88.57% 5 Missing and 3 partials ⚠️
infrahub_sdk/convert_object_type.py 84.00% 2 Missing and 2 partials ⚠️
infrahub_sdk/ctl/schema.py 77.77% 0 Missing and 2 partials ⚠️
infrahub_sdk/ctl/validate.py 33.33% 2 Missing ⚠️
infrahub_sdk/spec/object.py 94.44% 2 Missing ⚠️
infrahub_sdk/ctl/config.py 80.00% 1 Missing ⚠️
infrahub_sdk/playback.py 0.00% 1 Missing ⚠️
@@                 Coverage Diff                  @@
##           infrahub-develop     #552      +/-   ##
====================================================
+ Coverage             75.97%   76.06%   +0.08%     
====================================================
  Files                   100      102       +2     
  Lines                  9345     9073     -272     
  Branches               1835     1794      -41     
====================================================
- Hits                   7100     6901     -199     
+ Misses                 1754     1685      -69     
+ Partials                491      487       -4     
Flag Coverage Δ
integration-tests 34.97% <23.55%> (-1.21%) ⬇️
python-3.10 48.55% <54.66%> (-1.05%) ⬇️
python-3.11 48.55% <54.66%> (-1.03%) ⬇️
python-3.12 48.55% <54.66%> (-1.03%) ⬇️
python-3.13 48.55% <54.66%> (-1.03%) ⬇️
python-3.9 47.19% <53.77%> (-0.81%) ⬇️
python-filler-3.12 24.86% <12.00%> (+0.55%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
infrahub_sdk/branch.py 53.90% <100.00%> (-0.65%) ⬇️
infrahub_sdk/ctl/cli_commands.py 70.61% <100.00%> (+1.40%) ⬆️
infrahub_sdk/ctl/utils.py 68.53% <100.00%> (+0.44%) ⬆️
infrahub_sdk/diff.py 95.23% <100.00%> (-0.12%) ⬇️
infrahub_sdk/exceptions.py 91.07% <100.00%> (+9.25%) ⬆️
infrahub_sdk/graphql.py 99.11% <100.00%> (+<0.01%) ⬆️
infrahub_sdk/node/attribute.py 95.00% <100.00%> (+0.12%) ⬆️
infrahub_sdk/node/node.py 76.74% <100.00%> (+0.55%) ⬆️
infrahub_sdk/repository.py 100.00% <100.00%> (ø)
infrahub_sdk/timestamp.py 83.78% <100.00%> (+0.74%) ⬆️
... and 8 more

... and 9 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

ogenstad and others added 12 commits September 17, 2025 14:35
Merge 'stable' into 'develop' with resolved conflicts
* add support for clearing optional dropdown attributes and enhance tests for dropdown handling

* add changelog entry for clearing optional dropdown attributes

* refactor: update branch handling in changelog and allow clearing optional attributes if mutated
* add support for clearing optional dropdown attributes and enhance tests for dropdown handling

* add changelog entry for clearing optional dropdown attributes

* refactor: update branch handling in changelog and allow clearing optional attributes if mutated

Co-authored-by: Alex Gittings <[email protected]>
* Fix schema load failure exception

* Add towncrier housekeeping message
Merge stable into develop
* Add range expansion feature for object files and update documentation

- Implemented range expansion for string fields in object files, allowing patterns like [1-5] to create multiple objects.
- Added validation to ensure expanded lists have the same length.
- Updated documentation to include usage examples and details on the new feature.
- Added unit tests to verify range expansion functionality and error handling for mismatched lengths.

* Refactor range expansion tests to clarify expected behavior for edge cases

* Refactor range expansion logic to a standalone function for improved reusability and clarity

* Rename variable for clarity in data expansion process within InfrahubObjectFileData class

* Remove unnecessary range expansion in InfrahubObjectFileData class to streamline data handling
@ogenstad ogenstad merged commit cb4c330 into infrahub-develop Oct 7, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group/ci Issue related to the CI pipeline type/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants