Skip to content

Conversation

@thomasschafer
Copy link
Contributor

@thomasschafer thomasschafer commented Nov 20, 2025

  • Tests written and linted ℹ︎
  • Documentation written ℹ︎ - n/a
  • Commit history is tidy ℹ︎

What this does

This PR:

  • Passes the target file out of the uv plugin, replacing the hardcoded uv.lock
  • Does a little refactoring in preparation for passing excluded files out of the plugin, which will be any pyproject.toml and requirements.txt files in the uv project directory. Unfortunately we are not able to wire this all up yet, which is why a couple of lines are commented out, as the --exclude flag in the CLI only accepts file or directory names (e.g. requirements.txt or dir), and not full paths (e.g. dir/requirements.txt), which is not acceptable to us currently as do not want to add blanket excludes in this way - we need more targeted exclusions. This will be addressed in another story.

@snyk-io
Copy link

snyk-io bot commented Nov 20, 2025

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@thomasschafer thomasschafer changed the title chore: pass target file out of plugin chore: pass target file and exclusions out of plugin Nov 20, 2025
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch from 31bcc86 to ea6fc15 Compare November 21, 2025 11:09
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch 2 times, most recently from 2fbefbf to 7572701 Compare November 21, 2025 12:02
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch 2 times, most recently from 0ce82d4 to c0d5ab3 Compare November 21, 2025 15:34
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch 2 times, most recently from 6af74c1 to 3178baf Compare November 21, 2025 16:47
@thomasschafer thomasschafer marked this pull request as ready for review November 21, 2025 16:49
@thomasschafer thomasschafer requested review from a team as code owners November 21, 2025 16:49
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch from 3178baf to 88d4414 Compare November 21, 2025 16:55
@thomasschafer thomasschafer force-pushed the chore/pass-target-file-out-of-plugin branch from 88d4414 to c46066c Compare November 21, 2025 17:03
@thomasschafer thomasschafer merged commit 6dfcfee into main Nov 24, 2025
14 checks passed
@thomasschafer thomasschafer deleted the chore/pass-target-file-out-of-plugin branch November 24, 2025 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants