Skip to content

Move to standard pre-commit configuration where possible#1017

Merged
nathanjmcdougall merged 4 commits intomainfrom
1010-use-standard-pre-commit-config-except-for-deptry-and-import-linter
Oct 14, 2025
Merged

Move to standard pre-commit configuration where possible#1017
nathanjmcdougall merged 4 commits intomainfrom
1010-use-standard-pre-commit-config-except-for-deptry-and-import-linter

Conversation

@nathanjmcdougall
Copy link
Copy Markdown
Collaborator

Revert changes to uv-based pre-commits to avoid pinning uv

Revert changes to `uv`-based pre-commits to avoid pinning `uv`
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Oct 14, 2025

CodSpeed Performance Report

Merging #1017 will improve performances by 7.67%

Comparing 1010-use-standard-pre-commit-config-except-for-deptry-and-import-linter (2b1fbcf) with main (3d662cb)

Summary

⚡ 1 improvement
✅ 1 untouched

Benchmarks breakdown

Mode Benchmark BASE HEAD Change
Instrumentation test_several_tools_add_and_remove 1.3 s 1.2 s +7.67%

@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review October 14, 2025 20:37
@nathanjmcdougall nathanjmcdougall merged commit 236a596 into main Oct 14, 2025
20 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 1010-use-standard-pre-commit-config-except-for-deptry-and-import-linter branch October 14, 2025 20:37
@nathanjmcdougall nathanjmcdougall linked an issue Oct 14, 2025 that may be closed by this pull request
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.

Revisit declaring uv as a dev dep Use standard pre-commit config except for deptry and Import Linter Provide a way to add ruff without using uv

1 participant