Skip to content

Fix PEP 8 #9

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

Merged
merged 3 commits into from
Mar 5, 2021
Merged

Fix PEP 8 #9

merged 3 commits into from
Mar 5, 2021

Conversation

BobDotCom
Copy link
Member

@BobDotCom BobDotCom commented Mar 5, 2021

Description

Please include a summary of the change and which issue (if any) is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
All files updated for PEP 8, cleared by flake 8 linter

If this fixes an issue, please list it below.

Fixes #8

How to fill this out

Please mark options that are not relevant with the * character (like this*).
Replace the space in the box with an x to check the box (it should look like [x]) OR you may mark your changes immediately after submitting the PR.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (Please specify): Style fix

Has this been tested:

  • Yes
  • No
  • Not needed

Checklist:

If any are false, you may still submit, but may be asked to make changes.

  • My code follows the style guidelines of this project

  • I have changed/added the help doc at the beginning of any commands changed/added.

  • I have performed a self-review of my own code

  • I have commented my code, particularly in hard-to-understand areas

  • My changed files do not include any files regularly changed in saves (something where my bot would interfere with this).

  • My changes generate no new warnings

  • I have tested and concluded that my changes are effective or that my feature works - If you have not tested, dont check this so I know to review it more. You may still make a PR for untested code.

  • My code does not cause issues with existing code

  • Any dependent changes have been merged and published in downstream modules

  • Some of these options did not apply to my change (Please specify which):

@BobDotCom BobDotCom merged commit 956451e into main Mar 5, 2021
@BobDotCom BobDotCom deleted the fix branch March 5, 2021 16:29
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.

Make Code Better.
1 participant