fix: use uv tool run for pre-commit if not global
Metadata
Current evaluation
Merged as commit 55bf1ef; pre-commit setup now uses uv tool run when no global pre-commit executable is available.
Resolution: closed resolved
Reason: PR canonical/starbase#300 was approved by two reviewers and merged as commit 55bf1ef. It added a conditional fallback to uv tool run while preserving the global pre-commit path.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Issue found here: https://github.com/canonical/craft-store/pull/241#pullrequestreview-2502863941
- [ ] Have you followed the guidelines for contributing?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `tox`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as commit 55bf1ef; pre-commit setup now uses uv tool run when no global pre-commit executable is available. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #300 as commit 55bf1ef, making pre-commit run through uv tool run when it is not globally installed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #300 in commit 55bf1ef, fixing setup-precommit to use `uv tool run` when pre-commit is unavailable globally while retaining the global command otherwise. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR canonical/starbase#300 as commit 55bf1ef, using uv tool run for pre-commit when it is not installed globally. Approved by two reviewers. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #300 as commit 55bf1ef, updating setup to use uv tool run for pre-commit when no global executable is available. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix to use uv tool run for pre-commit when not installed globally. Approved by two reviewers with passing CI. Clarified that uv tracks tools independently, explaining the existing installation message. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Updates the setup process to use uv tool run for pre-commit when not installed globally. Maintainer confirmed the installation behavior and messages are correct, resolving reviewer concerns. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/craft-store#241
(confidence 95%)
A review there identified the original setup problem that prompted this fix.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #68 tools: add pre-commit | starbase | merged | Merged as commit 7a22e93, adding pre-commit tooling and configuration across three files. |