-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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] - implement C2701 import-private-name
#8903
Conversation
import-private-name
|
code | total | + violation | - violation | + fix | - fix |
---|---|---|---|---|---|
PLC2701 | 2522 | 2522 | 0 | 0 | 0 |
F401 | 2 | 2 | 0 | 0 | 0 |
de3ac71
to
40e0c24
Compare
61e04ae
to
ff4d6ff
Compare
There's an old PR (#5920) that implements this rule that hasn't been merged. |
Woop, I should have checked! I concede. 🤣 Thanks for the heads-up, @tjkuson! |
Well, your implementation might be better! Mine is quite old… |
Ah, I looked over yours, it's got things I didn't implement, I prefer yours! |
Summary
Implements
import-private-name/C2701
See: #970
Test Plan
cargo test
and manually