-
Notifications
You must be signed in to change notification settings - Fork 183
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
Pylint #620
Pylint #620
Conversation
Enabled remaining pylint defaults Signed-off-by: Sejal Verma <[email protected]>
@sejal-verma, DCO check is failing and also changelog entry is missing. |
Signed-off-by: Sejal Verma <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #620 +/- ##
==========================================
+ Coverage 72.09% 72.12% +0.02%
==========================================
Files 89 89
Lines 7939 7939
==========================================
+ Hits 5724 5726 +2
+ Misses 2215 2213 -2 ☔ View full report in Codecov by Sentry. |
Enabled remaining pylint defaults Signed-off-by: Sejal Verma <[email protected]> Signed-off-by: Sejal <[email protected]>
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.
- Does this actually enable them? I don't believe so, or you would have found many violations - AFAIK those get put in
[pylint.MESSAGE CONTROL]
. Try adding one and see the errors when you rnnox -rs lint
. - Sign DCO with
-s
for your commits.
# Conflicts: # setup.cfg
Signed-off-by: Sejal Verma <[email protected]>
Signed-off-by: Sejal Verma <[email protected]>
Signed-off-by: Sejal <[email protected]> Signed-off-by: Sejal Verma <[email protected]>
Signed-off-by: Sejal <[email protected]> Signed-off-by: Sejal Verma <[email protected]>
Enabled remaining pylint defaults Signed-off-by: Sejal <[email protected]>
# Conflicts: # setup.cfg
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.
You can see why I didn't enable all these lints in the first place. Will you be fixing the thousands of errors? Maybe start with one?
Signed-off-by: Sejal <[email protected]>
Signed-off-by: Sejal <[email protected]>
Signed-off-by: Sejal <[email protected]>
…arch-py into pylint_enable
Closing this, please reopen addressing #620 (review). |
Description
Enables lints in setup.cfg
Issues Resolved
Enables remaining pylint defaults
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.