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.

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

Introduce Black formatting, additional tests #623

Merged
merged 1 commit into from May 5, 2019
Merged

Introduce Black formatting, additional tests #623

merged 1 commit into from May 5, 2019

Conversation

phalt
Copy link
Contributor

@phalt phalt commented Apr 30, 2019 •
edited

This PR:

  • Introduces Black code formatting to the project.
  • Reformats the entire project against Black.
  • Adds Python 3.7 to the testing suite thanks to this suggestion.
  • Adds Django 2.2 compatibility testing
  • Pins versions of flake8 and pycopg2 in travis

@coveralls
Copy link

coveralls commented Apr 30, 2019 •
edited

Coverage Status

Coverage remained the same at 94.216% when pulling e6ad588 on black into 05c89c1 on master.

@phalt phalt force-pushed the black branch 2 times, most recently from 356cd56 to 48d2727 Compare April 30, 2019 09:20
@phalt phalt added this to In review in Improvements Apr 30, 2019
mvanlonden
mvanlonden reviewed Apr 30, 2019
View reviewed changes
setup.cfg Outdated Show resolved Hide resolved
@phalt
Copy link
Contributor Author

phalt commented May 2, 2019

@mvanlonden updated!

@phalt phalt merged commit 9a43f65 into master May 5, 2019
@phalt phalt deleted the black branch May 5, 2019 15:43
@phalt phalt moved this from In review to Done in Improvements May 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mvanlonden mvanlonden mvanlonden approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Improvements
  
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@phalt @coveralls @mvanlonden