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

Reference N817 from N814, and vice-versa #166

Merged
merged 1 commit into from May 1, 2021

Conversation

jparise
Copy link
Member

@jparise jparise commented Mar 26, 2021

Also add anchors and pep8.org references for many of the rules.

See #165

@sigmavirus24
Copy link
Member

I don't believe this explains the separate error codes which is what was requested in there by Sam

@jparise
Copy link
Member Author

jparise commented Mar 26, 2021

Yes, I totally agree. I was thinking this could be a useful small improvement.

But if a larger documentation section explaining the codes is in the works, then that's even better.

README.rst Outdated Show resolved Hide resolved
@jparise
Copy link
Member Author

jparise commented Apr 13, 2021

I took another pass on this section, adding pep8.org references where they were obvious, as well.

README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
README.rst Outdated Show resolved Hide resolved
Also add PEP 8 references for many of the rules.
@sigmavirus24 sigmavirus24 merged commit 2db856d into PyCQA:master May 1, 2021
@jparise jparise deleted the README-N814-N817 branch May 1, 2021 16:18
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

Successfully merging this pull request may close these issues.

None yet

2 participants