-
-
Notifications
You must be signed in to change notification settings - Fork 67
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
Added sp-repo-review to pre-commit #411
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #411 +/- ##
==========================================
- Coverage 93.62% 93.60% -0.03%
==========================================
Files 25 25
Lines 894 891 -3
==========================================
- Hits 837 834 -3
Misses 57 57 ☔ View full report in Codecov by Sentry. |
2feff68
to
3e41606
Compare
Note to self: I need to update the path to the coverage config file in tox.ini |
16aae87
to
7acb224
Compare
7acb224
to
cd88ebe
Compare
cd88ebe
to
ff8d5f2
Compare
@neutrinoceros - would you be happy to review this as you suggested? |
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.
LGTM, I have nothing of value to add here !
Just exploring sp-repo-review and seeing if it provides useful feedback
Fixes #421