ci(tests) Verify runtime deps - #45
Conversation
Reviewer's Guide by SourceryThis pull request adds a CI check to verify runtime dependencies using No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey @tony - I've reviewed your changes - here's some feedback:
Overall Comments:
- Consider adding a step to verify that the installed runtime dependencies match the declared dependencies in
pyproject.toml.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
|
Automated review finding. Recorded here so it is not lost. Nothing has been pushed to this branch and no fix is applied — this is a note, not a change request. CI imports
|
Changes
uv run --no-devCHANGESSummary by Sourcery
Add a CI check to verify runtime dependencies before installing dev dependencies, including a basic functionality test.
CI:
uv run --no-devbefore installing dev dependencies.Tests: