Skip to content
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 suppression comments trigger new lint #103

Open
jml opened this issue Jun 12, 2015 · 1 comment
Open

pylint suppression comments trigger new lint #103

jml opened this issue Jun 12, 2015 · 1 comment

Comments

@jml
Copy link
Member

jml commented Jun 12, 2015

I have the line

    def global_tcpip_forward(self, data):  # pylint: disable=C0103

Which causes the following warning to be emitted:

W9402: 88,0: The first letter of comment should be capitalized

This is silly. I would suggest dropping the check altogether.

@wsanchez
Copy link
Member

wsanchez commented Jun 4, 2016

I agree. This check provides no real benefit. Even when we're dealing with actual words, are the docs really so much more readable when one writes:

foo = 4  # A counter

versus:

foo = 4  # a counter

Is it really worth out time to be chasing this nonsense down? And I'm often starting a comment with the name of a symbol, so to work around this, I need to rephrase my comment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants