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

Disable E501 for inline comments #7400

Closed
Actticus opened this issue Sep 15, 2023 · 4 comments
Closed

Disable E501 for inline comments #7400

Actticus opened this issue Sep 15, 2023 · 4 comments
Labels
needs-decision Awaiting a decision from a maintainer rule Implementing or modifying a lint rule

Comments

@Actticus
Copy link

Our project has pytest-cov, and some lines are disabled from being checked. We use the # noqa: nocover comment, and sometimes it goes beyond the length of the line. I would like to disable the E501 error for inline comments, is there any way to do this?

@charliermarsh
Copy link
Member

Do you want E501 to be disabled for all inline comments, or only pragma comments like # noqa?

@charliermarsh charliermarsh added the needs-info More information is needed from the issue author label Sep 19, 2023
@Actticus
Copy link
Author

I think it's better for all inline comments

@dhruvmanila dhruvmanila added needs-decision Awaiting a decision from a maintainer rule Implementing or modifying a lint rule and removed needs-info More information is needed from the issue author labels Sep 25, 2023
@charliermarsh
Copy link
Member

For now, we are disabling E501 for pragma comments, but not for all inline comments (#7692).

@Actticus
Copy link
Author

Actticus commented Oct 1, 2023

Ty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-decision Awaiting a decision from a maintainer rule Implementing or modifying a lint rule
Projects
None yet
Development

No branches or pull requests

3 participants