ci(tests) Verify runtime deps - #460
Conversation
Reviewer's Guide by SourceryThis pull request adds a runtime dependency check to the CI workflow 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 the package installs without errors using
pip install .oruv pip install ..
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.
f4acb11 to
02d4ed0
Compare
02d4ed0 to
107432e
Compare
107432e to
60edf55
Compare
|
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 references
|
Changes
uv run --no-devCHANGESSee also
Summary by Sourcery
Adds a CI job to verify runtime dependencies before installing dev dependencies, ensuring the package functions correctly with its declared runtime dependencies.
CI:
uv run --no-devbefore installing dev dependencies.Tests: