-
Notifications
You must be signed in to change notification settings - Fork 55
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
Pre commit #167
Pre commit #167
Conversation
Check code and docs spelling via `pre-commit`
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #167 +/- ##
=======================================
Coverage 90.34% 90.34%
=======================================
Files 23 23
Lines 2393 2393
=======================================
Hits 2162 2162
Misses 231 231
☔ View full report in Codecov by Sentry. |
@rhugonnet I finally found the time to do the The check is now passing, so we can continue from here. Do you want to add more checks on this branch, before merging? |
Amazing, great additions :) |
Continuation of #163
I merged into a non-main branch to push some stuff before merging into main