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
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