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

updated code to ensure pep8 compliance #17

Merged
merged 1 commit into from
Oct 18, 2020
Merged

updated code to ensure pep8 compliance #17

merged 1 commit into from
Oct 18, 2020

Conversation

sculley
Copy link
Contributor

@sculley sculley commented Oct 18, 2020

I have updated the code to address the PEP8 validation warnings/errors found by flake8 identified in [Issue #4]

bot.py:4:1: F401 'json' imported but unused
bot.py:12:1: E302 expected 2 blank lines, found 1
bot.py:13:80: E501 line too long (130 > 79 characters)
bot.py:16:1: E305 expected 2 blank lines after class or function definition, found 1
bot.py:18:1: E265 block comment should start with '# '
bot.py:19:17: E231 missing whitespace after ','
bot.py:22:1: E302 expected 2 blank lines, found 1
bot.py:26:1: E302 expected 2 blank lines, found 1
bot.py:31:21: E231 missing whitespace after ','
bot.py:33:25: E231 missing whitespace after ','
bot.py:36:17: E265 block comment should start with '# '
bot.py:36:80: E501 line too long (96 > 79 characters)
bot.py:40:41: E251 unexpected spaces around keyword / parameter equals
bot.py:40:43: E251 unexpected spaces around keyword / parameter equals
bot.py:53:21: E231 missing whitespace after ','
bot.py:62:1: E303 too many blank lines (3)
bot.py:69:1: E305 expected 2 blank lines after class or function definition, found 1

@sculley sculley mentioned this pull request Oct 18, 2020
@TaniaMalhotra
Copy link
Owner

This looks perfect
Thank you for your contribution @sculley 🎉

@TaniaMalhotra
Copy link
Owner

Do star the repository if you found my bot useful :)

@TaniaMalhotra TaniaMalhotra merged commit c581b83 into TaniaMalhotra:master Oct 18, 2020
@TaniaMalhotra TaniaMalhotra linked an issue Oct 18, 2020 that may be closed by this pull request
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.

Code refactoring
2 participants