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

Change variable in README according to the PEP8 #1002

Merged
merged 2 commits into from
Sep 4, 2019
Merged

Change variable in README according to the PEP8 #1002

merged 2 commits into from
Sep 4, 2019

Conversation

vezeli
Copy link
Contributor

@vezeli vezeli commented Aug 29, 2019

Black is constantly aiming to follow PEP 8 and this is nicely put in the README. However, PEP 8 also suggests

Never use the characters 'l' (lowercase letter el), 'O' (uppercase letter oh), or 'I' (uppercase letter eye) as single character variable names.

For this reason I have made a micro alteration and changed the variable letter l in an example to another variable in the spirit of PEP 8.

@zsol zsol merged commit c4d2db4 into psf:master Sep 4, 2019
@zsol
Copy link
Collaborator

zsol commented Sep 4, 2019

Thanks!

@vezeli vezeli deleted the pep8readme branch September 4, 2019 09:27
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.

3 participants