Skip to content

Conversation

@infrahub-github-bot-app
Copy link
Contributor

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

Replace ternary `if` expression with `or` operator
Prefer variables in lowercase
Simplify for-loop with any()
Prevent cascade delete from causing SDK tracking delete to fail
Merge stable into develop
Use a list comprehension to create a transformed list
Add missing __init__.py files
Fix ruff Jinja2 violations and move the remaining warning
Add __hash__ method to object that already has __eq__
Update packages: urllib3, filelock & virtualenv
Fix flake8 raise linting violations
Avoid shadowing builtin `list`
Merge stable into develop
@infrahub-github-bot-app infrahub-github-bot-app bot requested a review from a team as a code owner January 28, 2026 15:59
@infrahub-github-bot-app infrahub-github-bot-app bot self-assigned this Jan 28, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 28, 2026

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.

  • 🔍 Trigger a full review

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: 4843e01
Status:🚫  Deploy failed.

View logs

@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 60.46512% with 17 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/query_groups.py 16.66% 5 Missing ⚠️
infrahub_sdk/client.py 25.00% 3 Missing ⚠️
infrahub_sdk/ctl/cli_commands.py 0.00% 3 Missing ⚠️
infrahub_sdk/ctl/check.py 0.00% 2 Missing ⚠️
infrahub_sdk/ctl/config.py 0.00% 2 Missing ⚠️
infrahub_sdk/generator.py 0.00% 1 Missing ⚠️
infrahub_sdk/recorder.py 50.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (d6dc732) and HEAD (4843e01). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (d6dc732) HEAD (4843e01)
integration-tests 1 0
@@                 Coverage Diff                  @@
##           infrahub-develop     #787      +/-   ##
====================================================
- Coverage             80.39%   72.33%   -8.07%     
====================================================
  Files                   115      115              
  Lines                  9872     9865       -7     
  Branches               1513     1504       -9     
====================================================
- Hits                   7937     7136     -801     
- Misses                 1413     2223     +810     
+ Partials                522      506      -16     
Flag Coverage Δ
integration-tests ?
python-3.10 51.38% <55.81%> (-0.06%) ⬇️
python-3.11 51.36% <55.81%> (-0.08%) ⬇️
python-3.12 51.38% <55.81%> (-0.06%) ⬇️
python-3.13 51.38% <55.81%> (-0.08%) ⬇️
python-3.14 53.03% <55.81%> (-0.08%) ⬇️
python-filler-3.12 24.07% <4.65%> (+0.02%) ⬆️

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

Files with missing lines Coverage Δ
infrahub_sdk/batch.py 94.36% <100.00%> (-0.16%) ⬇️
infrahub_sdk/ctl/repository.py 72.00% <100.00%> (ø)
infrahub_sdk/ctl/utils.py 68.53% <100.00%> (ø)
infrahub_sdk/diff.py 96.42% <100.00%> (-0.13%) ⬇️
infrahub_sdk/graphql/renderers.py 90.47% <100.00%> (-2.39%) ⬇️
infrahub_sdk/protocols_generator/generator.py 95.00% <100.00%> (ø)
infrahub_sdk/schema/repository.py 84.35% <100.00%> (-2.98%) ⬇️
infrahub_sdk/spec/object.py 59.21% <100.00%> (-26.09%) ⬇️
infrahub_sdk/utils.py 86.17% <100.00%> (-2.36%) ⬇️
infrahub_sdk/yaml.py 81.65% <100.00%> (-4.72%) ⬇️
... and 7 more

... and 22 files with indirect coverage changes

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant