Pin the pre-commit hook hashes#294
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #294 +/- ##
=======================================
Coverage 79.19% 79.19%
=======================================
Files 51 51
Lines 5542 5542
Branches 577 580 +3
=======================================
Hits 4389 4389
Misses 1153 1153
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: pdmurray <peynmurray@gmail.com> Co-authored-by: Sarah Monod <smonod1@bloomberg.net>
ea5ec26 to
7a5db2b
Compare
Signed-off-by: pdmurray <peynmurray@gmail.com>
|
It looks like the newly updated pre-commit hooks resulted in a few formatting changes, some of which hit example python files used by tests. This means that there are some line number changes for a few of the integration tests. These should now be up to date with the latest changes 👍 |
|
Does it make sense to write a news entry for this? |
Nope, I've added the skip news label for it. |
Issue number of the reported bug or feature request: Closes #293.
Describe your changes
This PR pins (and updates) the pre-commit hooks using commit hashes rather than (potentially mutable) git refs. For future reference, the command used here was
pre-commit autoupdate --freeze.Testing performed
No testing required.