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.

Sign up for GitHub

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

Jump to bottom

Cleanup #4

Merged

Ziyadsk merged 4 commits into Ziyadsk:master from CosmicHorrorDev:cleanup

Oct 16, 2018

Contributor

CosmicHorrorDev commented Oct 16, 2018

This pull request does a few minor things to improve readability and consistency.

Follows pylint and flake8 standards for:
- Stripping trailing whitespace
- Having a space follow commas when passing parameters
- Sorted imports
More consistent use of " for surrounding strings
Reduced nesting by moving code out of the with block when possible
Restructuring more of the for-loops to use .items() to access vals from dicts
Small other bits of cleanup

CosmicHorrorDev added 4 commits

October 16, 2018 12:04


          Linted with pylint and flake8

4d3c4bf


          Improve consistency

fdcb69f


          Reduce nesting

1ad6ad1


          Use .items() on for-loops

e7ff0ad

Ziyadsk merged commit 1bc47c6 into Ziyadsk:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet