Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Update the setup.py #64

Merged
merged 3 commits into from
Oct 26, 2020
Merged

Update the setup.py #64

merged 3 commits into from
Oct 26, 2020

Conversation

GreatBahram
Copy link
Contributor

  • sort imported modules
  • use dedent function to indent the multi-line string

@GreatBahram
Update the setup.py
c67a5b0 
* sort imported modules
* use dedent to indent multi-line string
@pep8speaks
Copy link

pep8speaks commented Oct 25, 2020 •
edited
Loading

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

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-10-26 18:50:28 UTC

@GreatBahram
Copy link
Contributor Author

I checked the Travis CI problems, those are not mine. Maybe you want to drop legacy Python and get rid of them.

@rasbt
Copy link
Owner

rasbt commented Oct 26, 2020

Thanks for the PR!

@rasbt rasbt merged commit d086295 into rasbt:master Oct 26, 2020
@GreatBahram
Copy link
Contributor Author

You're welcome

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GreatBahram @pep8speaks @rasbt