Skip to content

Generalize frozen pre-commit install instruction for uv users#762

Merged
nathanjmcdougall merged 2 commits into
mainfrom
759-in-install_pre_commit_hooks-ensure-the-message-adapts-based-on-uv-usage
Jun 18, 2025
Merged

Generalize frozen pre-commit install instruction for uv users#762
nathanjmcdougall merged 2 commits into
mainfrom
759-in-install_pre_commit_hooks-ensure-the-message-adapts-based-on-uv-usage

Conversation

@nathanjmcdougall

Copy link
Copy Markdown
Collaborator

No description provided.

@codspeed-hq

codspeed-hq Bot commented Jun 18, 2025

Copy link
Copy Markdown

CodSpeed Walltime Performance Report

Merging #762 will not alter performance

Comparing 759-in-install_pre_commit_hooks-ensure-the-message-adapts-based-on-uv-usage (c55949e) with main (ebdff2b)

Summary

✅ 1 untouched benchmarks

@codecov

codecov Bot commented Jun 18, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 75.00000% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/usethis/_integrations/pre_commit/core.py 75.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@nathanjmcdougall nathanjmcdougall merged commit e8d49e0 into main Jun 18, 2025
18 of 19 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 759-in-install_pre_commit_hooks-ensure-the-message-adapts-based-on-uv-usage branch June 18, 2025 10:05
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.

In install_pre_commit_hooks, ensure the message adapts based on uv usage

1 participant