⬆️🧰 Autoupdate pre-commit config #1438
Conversation
s-weigand
requested review from
jsnel,
a team and
joernweissenborn
as code owners
|
|
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hey @s-weigand - I've reviewed your changes and they look great!
General suggestions:
- Ensure that the updated pre-commit hooks do not introduce any unintended formatting changes.
- Verify that all dependencies updated in the pre-commit config are compatible with the project's current setup.
- Consider running the full test suite to ensure that the updates do not affect the project's functionality.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Docstrings: all looks good
Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## staging #1438 +/- ##
=========================================
- Coverage 84.9% 84.9% -0.1%
=========================================
Files 91 91
Lines 3742 3740 -2
Branches 728 728
=========================================
- Hits 3178 3176 -2
+ Misses 450 448 -2
- Partials 114 116 +2 ☔ View full report in Codecov by Sentry. |
Yeah pretty much the same as #1407 on |
Note this is an update for the staging branch. * 🧰⬆️ Update pre-commit config * 🧹 Run pre-commit on all files
Minor tooling update
Change summary
Checklist