-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: update pre-commit hooks #571
Conversation
acda0be
to
48ded74
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #571 +/- ##
=======================================
Coverage 58.40% 58.40%
=======================================
Files 62 62
Lines 6433 6433
Branches 1150 1150
=======================================
Hits 3757 3757
Misses 2385 2385
Partials 291 291 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
48ded74
to
e34bef4
Compare
NOTE:re-spun checks, and the error on the main tests has now passed off. |
f6b5e25
to
c5ca3f6
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Templating
Version numbers are not typically covered by templating. It is expected that this PR is 100% about advancing version numbers, which would not require any templating follow-up. Please double-check for any other changes that might be suitable for templating.
updates: - [github.com/astral-sh/ruff-pre-commit: v0.9.10 → v0.11.2](astral-sh/ruff-pre-commit@v0.9.10...v0.11.2) - [github.com/abravalheri/validate-pyproject: v0.23 → v0.24.1](abravalheri/validate-pyproject@v0.23...v0.24.1)
c5ca3f6
to
a7e828c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see SciTools/.github#77
Changes were just merging in PR, didn't affect actual template
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK now I believe!
Thanks to @ESadek-MO SciTools/.github#78
updates: