-
Notifications
You must be signed in to change notification settings - Fork 301
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
Configure Codacy from file to reproduce same error messages locally #1804
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1804 +/- ##
=======================================
Coverage 69.75% 69.75%
=======================================
Files 148 148
Lines 18610 18610
=======================================
Hits 12982 12982
Misses 5628 5628 |
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.
+1 We should investigate why codacy is not running on this branch
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.
Wow, that looks like a big work! Big thanks, Dominik! I went through it quickly assuming that this is a port of current codacy settings (i think in the future we might want to remove quite a few of entries from disable
section :) )
please Mikhail, remove some disable settings. I started with the original settings and already removed some, of which I thought they were purely beneficial to remove. |
Currently the only python complaints raised by codacy stem from pylint. The standard config of pylint is massively tweaked via a filter set in the codacy ui. In this PR I went through some painful regexps to convert the html of the config page to a useful
.pylintrc
file.This file can be used instead of the UI by codacy, but it can also be used by running pylint locally.