Skip to content

Conversation

@ogenstad
Copy link
Contributor

@ogenstad ogenstad commented Feb 9, 2026

Summary by CodeRabbit

Release Notes

  • Bug Fixes

    • Enhanced error handling for Jinja template syntax errors with detailed error messages.
  • Chores

    • Updated development dependencies and tools to latest versions.
    • Added configuration for automated dependency updates.
    • Enhanced linting rules and pre-commit configuration.
  • Tests

    • Added comprehensive test coverage for template syntax error handling.

ogenstad and others added 8 commits February 4, 2026 17:09
Raise SDK specific Jinja errors on syntax error
Signed-off-by: Fatih Acar <fatih@opsmill.com>
Bumps [infrahub-testcontainers](https://github.com/opsmill/infrahub) from 1.5.1 to 1.7.4.
- [Release notes](https://github.com/opsmill/infrahub/releases)
- [Changelog](https://github.com/opsmill/infrahub/blob/stable/CHANGELOG.md)
- [Commits](opsmill/infrahub@infrahub-v1.5.1...infrahub-v1.7.4)

---
updated-dependencies:
- dependency-name: infrahub-testcontainers
  dependency-version: 1.7.4
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
…stcontainers-1.7.4

Bump infrahub-testcontainers from 1.5.1 to 1.7.4
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Walkthrough

This pull request updates development dependencies and enhances template error handling. Ruff is upgraded from version 0.14.10 to 0.15.0 in both pre-commit configuration and project dependencies, with a new Ruff rule RUF067 added to the ignore list. Dependabot configuration is extended to support the uv package ecosystem with an allow rule for a specific dependency. Template parsing in the SDK now wraps environment parsing calls with try/except blocks to catch and handle Jinja template syntax errors consistently. Corresponding unit tests are added to verify syntax error handling in template validation and variable extraction functions.

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title 'Merge stable into develop with resolved conflicts' describes a merge operation between branches, but the actual changes include template error handling, linting tool updates, and test additions—not a simple merge. Update the title to reflect the actual main change, such as 'Add Jinja template syntax error handling and update linting configuration' or 'Improve template syntax error handling with centralized error reporting'.
Docstring Coverage ⚠️ Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions bot added the group/ci Issue related to the CI pipeline label Feb 9, 2026
@cloudflare-workers-and-pages
Copy link

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: 95b9cf6
Status: ✅  Deploy successful!
Preview URL: https://066b38e1.infrahub-sdk-python.pages.dev
Branch Preview URL: https://pog-stable-to-develop-202602.infrahub-sdk-python.pages.dev

View logs

@codecov
Copy link

codecov bot commented Feb 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@             Coverage Diff             @@
##           develop     #809      +/-   ##
===========================================
- Coverage    81.52%   80.34%   -1.19%     
===========================================
  Files          117      115       -2     
  Lines        11822     9873    -1949     
  Branches      2025     1504     -521     
===========================================
- Hits          9638     7932    -1706     
+ Misses        1541     1420     -121     
+ Partials       643      521     -122     
Flag Coverage Δ
integration-tests 41.41% <0.00%> (-1.63%) ⬇️
python-3.10 51.40% <100.00%> (-2.78%) ⬇️
python-3.11 51.38% <100.00%> (-2.80%) ⬇️
python-3.12 51.40% <100.00%> (-2.78%) ⬇️
python-3.13 51.40% <100.00%> (-2.78%) ⬇️
python-3.14 53.05% <100.00%> (-3.19%) ⬇️
python-filler-3.12 24.06% <0.00%> (+0.34%) ⬆️

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

Files with missing lines Coverage Δ
infrahub_sdk/template/__init__.py 95.55% <100.00%> (+0.20%) ⬆️

... and 12 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 ogenstad marked this pull request as ready for review February 9, 2026 09:14
@ogenstad ogenstad requested a review from a team as a code owner February 9, 2026 09:14
@ogenstad ogenstad merged commit c29ede9 into develop Feb 9, 2026
21 checks passed
@ogenstad ogenstad deleted the pog-stable-to-develop-20260209 branch February 9, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude-code-assisted group/ci Issue related to the CI pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants