Skip to content

Conversation

@infrahub-github-bot-app
Copy link
Contributor

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

minitriga and others added 2 commits October 9, 2025 10:28
* Add data processing strategy for object files with range expansion support

- Introduced `strategy` field in object file specification to control data processing.
- Implemented `DataProcessor` classes for normal and range expansion strategies.
- Updated validation and processing methods to utilize the specified strategy.
- Enhanced tests to cover new strategy functionality and ensure correct behavior.

* Remove unused strategy parameter from InfrahubObjectFileData class

* Improve error handling for unknown data processing strategies and add test for invalid object expansion strategy

* Rename test_invalid_object_expansion_strategy to test_invalid_object_expansion_processor and add additional test for invalid strategy handling
…#574)

* IHS-170: Disable rich console print markup causing regex reformatting

* update tests
@github-actions github-actions bot added the type/documentation Improvements or additions to documentation label Oct 10, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 10, 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.

@cloudflare-workers-and-pages
Copy link

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: cf599ac
Status: ✅  Deploy successful!
Preview URL: https://4ada5f04.infrahub-sdk-python.pages.dev
Branch Preview URL: https://develop.infrahub-sdk-python.pages.dev

View logs

@codecov
Copy link

codecov bot commented Oct 10, 2025

Codecov Report

❌ Patch coverage is 89.36170% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/spec/object.py 88.63% 5 Missing ⚠️
@@                 Coverage Diff                  @@
##           infrahub-develop     #575      +/-   ##
====================================================
+ Coverage             76.31%   76.35%   +0.04%     
====================================================
  Files                   102      102              
  Lines                 10051    10086      +35     
  Branches               2077     2081       +4     
====================================================
+ Hits                   7670     7701      +31     
- Misses                 1827     1831       +4     
  Partials                554      554              
Flag Coverage Δ
integration-tests 36.20% <74.46%> (+0.13%) ⬆️
python-3.10 50.24% <85.10%> (+0.13%) ⬆️
python-3.11 50.22% <85.10%> (+0.11%) ⬆️
python-3.12 50.22% <85.10%> (+0.13%) ⬆️
python-3.13 50.22% <85.10%> (+0.13%) ⬆️
python-3.9 48.70% <85.10%> (+0.13%) ⬆️
python-filler-3.12 24.08% <0.00%> (-0.09%) ⬇️

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

Files with missing lines Coverage Δ
infrahub_sdk/ctl/schema.py 57.03% <100.00%> (ø)
infrahub_sdk/spec/range_expansion.py 88.57% <100.00%> (ø)
infrahub_sdk/spec/object.py 86.00% <88.63%> (+0.29%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dgarros dgarros merged commit f238ec6 into infrahub-develop Oct 11, 2025
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants