Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

new pylint rules plan #1638

Closed
leckie-chn opened this issue Oct 21, 2019 · 2 comments
Closed

new pylint rules plan #1638

leckie-chn opened this issue Oct 21, 2019 · 2 comments
Assignees
@leckie-chn
Labels
nnidev

Comments

@leckie-chn
Copy link
Contributor

reference: Pylint Documentation

  • On CI pipeline, by default enable all Error(E), Fatal(F) check, along with the following rules:
    W0614, C0326, W0611, C0330, C0411, C0303, W1201, C0301, W0612, W0401, W0235, C1801, W1202, W0622, W1505
  • C0111 will be enabled later
  • E1101, R0912, W0212 will not be enforced on pipeline, but recommend contributors comply their coding style to these rules.
@leckie-chn leckie-chn self-assigned this Oct 21, 2019
@leckie-chn
Copy link
Contributor Author

related PR: #1643

@liuzhe-lz
Copy link
Contributor

Done in PR #1697

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@leckie-chn leckie-chn

Labels
nnidev
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@leckie-chn @scarlett2018 @liuzhe-lz