Closed JasperCraeghs closed 5 years ago
Fix linelength of pep8 to 120 (or remove that check). There is also some problems with pylint...
Merging #36 into master will increase coverage by
0.17%
. The diff coverage is13.04%
.
@@ Coverage Diff @@
## master #36 +/- ##
==========================================
+ Coverage 13.59% 13.77% +0.17%
==========================================
Files 1 1
Lines 331 334 +3
==========================================
+ Hits 45 46 +1
- Misses 286 288 +2
Impacted Files | Coverage Δ | |
---|---|---|
mlx/coverity.py | 13.77% <13.04%> (+0.17%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 9704248...1ffe52a. Read the comment docs.
Don't overstreatch this PR with fixes. I would say we wrap it, remove the wip and merge. Then solve issues in another mr
The last commit regarding coverity_services.py should solve why the example didn't build properly last time I built it, but I can't confirm this yet without VPN.
I am merging this because it is going out of scope.
Add bandit, git-legal, pylint and pep8 plugins, and increase threshold where it makes sense.