-
Notifications
You must be signed in to change notification settings - Fork 101
[skip travis] DEV: Remove Non-Blocking Pre-Commit Checks #168
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
Conversation
|
@shaneahmed Please review and merge if you are happy with this. |
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 can you add how to ignore code complexity in the description for this PR?
We can then merge it.
- Set max-cognitive-complexity to 11
- Fix opencv resize error with v4.5.4
- Revert opencv bug fix
Codecov Report
@@ Coverage Diff @@
## develop #168 +/- ##
===========================================
- Coverage 99.92% 99.82% -0.11%
===========================================
Files 40 40
Lines 2751 3382 +631
Branches 458 605 +147
===========================================
+ Hits 2749 3376 +627
Misses 1 1
- Partials 1 5 +4
Continue to review full report at Codecov.
|
Make spelling checks a blocking pre-commit issue again and sort pre-commit flake8 dependency lines to be alphabetical