Skip to content

Use uv tool run for pre-commit to avoid dependency declaration#6

Merged
MrLYC merged 2 commits intodevin/1770481134-cleanup-poetry-filesfrom
copilot/sub-pr-4
Feb 7, 2026
Merged

Use uv tool run for pre-commit to avoid dependency declaration#6
MrLYC merged 2 commits intodevin/1770481134-cleanup-poetry-filesfrom
copilot/sub-pr-4

Conversation

Copy link
Contributor

Copilot AI commented Feb 7, 2026

The Makefile runs uv run pre-commit, but pre-commit isn't in pyproject.toml dependencies, causing failures in fresh environments.

Changes

  • Makefile: Replace uv run pre-commit with uv tool run pre-commit in install and check targets

This runs pre-commit in an isolated environment without requiring it as a project dependency.

# Before
@uv run pre-commit run -a

# After
@uv tool run pre-commit run -a

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: MrLYC <6391488+MrLYC@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback on removing Poetry files and uv migration Use uv tool run for pre-commit to avoid dependency declaration Feb 7, 2026
Copilot AI requested a review from MrLYC February 7, 2026 16:58
@MrLYC MrLYC marked this pull request as ready for review February 7, 2026 17:08
@MrLYC MrLYC merged commit e090ad9 into devin/1770481134-cleanup-poetry-files Feb 7, 2026
6 checks passed
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.

2 participants