Conversation
WalkthroughThis PR updates the project configuration file by bumping the 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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. Comment |
Deploying infrahub-sdk-python with
|
| Latest commit: |
376d5dd
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://2e9a4b06.infrahub-sdk-python.pages.dev |
| Branch Preview URL: | https://pog-upgrade-ty.infrahub-sdk-python.pages.dev |
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## stable #797 +/- ##
=======================================
Coverage 80.36% 80.36%
=======================================
Files 115 115
Lines 9865 9865
Branches 1504 1504
=======================================
Hits 7928 7928
Misses 1415 1415
Partials 522 522
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Actionable comments posted: 0
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (1)
pyproject.toml (1)
133-140:⚠️ Potential issue | 🟡 MinorConsider narrowing the
unused-ignore-commentsuppression scope or disabling only for# type: ignorecomments.Suppressing this rule globally disables detection of stale ignore comments across the codebase. ty does not offer a mypy-compat mode, but you can use
analysis.respect-type-ignore-comments = falseto prevent ty from treating# type: ignorecomments as suppression (useful if mypy is also in use). Otherwise, narrowing the scope via per-rule or per-file configuration would keep the signal without blanket suppression.
Upgrade ty and also add some additional ignore lines due to conflicts between ty and mypy at the moment.
Summary by CodeRabbit