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

Auto format code #217

Closed
wants to merge 7 commits into from
Closed

Auto format code #217

wants to merge 7 commits into from

Conversation

thomasyu888
Copy link
Member

@thomasyu888 thomasyu888 commented Mar 6, 2021

  • Fixes #xxxx (Link the github issue if applicable)
  • Did you run your tests locally (instructions here)?
  • Did you add a good description about your changes or additions?

Use black to auto format code

@pep8speaks
Copy link

pep8speaks commented Mar 6, 2021

Hello @thomasyu888! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 174:80: E501 line too long (82 > 79 characters)

Line 39:80: E501 line too long (278 > 79 characters)
Line 41:80: E501 line too long (103 > 79 characters)
Line 45:80: E501 line too long (91 > 79 characters)

Line 19:1: E302 expected 2 blank lines, found 1

Line 7:80: E501 line too long (85 > 79 characters)
Line 8:80: E501 line too long (128 > 79 characters)

Line 95:80: E501 line too long (85 > 79 characters)
Line 96:80: E501 line too long (88 > 79 characters)

Line 45:80: E501 line too long (84 > 79 characters)

Comment last updated at 2021-03-06 15:50:55 UTC

@thomasyu888 thomasyu888 closed this Aug 7, 2021
@thomasyu888 thomasyu888 deleted the black branch August 7, 2021 07:24
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