-
Notifications
You must be signed in to change notification settings - Fork 669
chore: add uv skeleton plugin #6511
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
cf9d060 to
5406945
Compare
0f4b27b to
876e8ac
Compare
| plugin: { | ||
| name: 'snyk-uv-plugin', | ||
| runtime: process.version, | ||
| targetFile, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we double check whether the target file for uv should be pyproject.toml or uv.lock
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checked and using Poetry we see pyproject.toml in the UI - will make a ticket to address
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is where some of the complexity occurs, where pyproject.toml is package manager agnostic and requires the companion poetry.local and uv.lock to differentiate.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
35344b8 to
6682393
Compare
6682393 to
bab2d4f
Compare
Pull Request Submission Checklist
are release-note ready, emphasizing
what was changed, not how.
What does this PR do?
This PR adds a minimal uv plugin, which just returns a fixed dep-graph. For now it is not wired up to anything so there will be no functional changes from this PR.
How should this be manually tested?
For now it cannot be tested end-to-end, as there is still work to do to hook everything up, but you can review the unit tests.
What's the product update that needs to be communicated to CLI users?
None