⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
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 67.44186% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
infrahub_sdk/client.py 25.00% 3 Missing ⚠️
infrahub_sdk/ctl/cli_commands.py 0.00% 3 Missing ⚠️
infrahub_sdk/query_groups.py 50.00% 3 Missing ⚠️
infrahub_sdk/ctl/check.py 0.00% 2 Missing ⚠️
infrahub_sdk/ctl/config.py 0.00% 2 Missing ⚠️
infrahub_sdk/recorder.py 50.00% 1 Missing ⚠️
@@                 Coverage Diff                  @@
##           infrahub-develop     #787      +/-   ##
====================================================
- Coverage             80.39%   80.36%   -0.04%     
====================================================
  Files                   115      115              
  Lines                  9872     9865       -7     
  Branches               1513     1504       -9     
====================================================
- Hits                   7937     7928       -9     
- Misses                 1413     1415       +2     
  Partials                522      522              
Flag Coverage Δ
integration-tests 41.41% <41.86%> (+0.01%) ⬆️
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/generator.py 79.06% <100.00%> (ø)
infrahub_sdk/graphql/renderers.py 92.85% <100.00%> (ø)
infrahub_sdk/protocols_generator/generator.py 95.00% <100.00%> (ø)
infrahub_sdk/schema/repository.py 87.07% <100.00%> (-0.26%) ⬇️
infrahub_sdk/spec/object.py 85.19% <100.00%> (-0.10%) ⬇️
infrahub_sdk/utils.py 88.47% <100.00%> (-0.06%) ⬇️
... and 7 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ogenstad ogenstad merged commit 424e0ca into infrahub-develop Jan 29, 2026
47 of 49 checks passed
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