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

Add Python formatter GitHub Action. #489

Merged
merged 3 commits into from Jun 26, 2020
Merged

Add Python formatter GitHub Action. #489

merged 3 commits into from Jun 26, 2020

Conversation

@ruebot
Copy link
Member

ruebot commented Jun 26, 2020

GitHub issue(s): #488

What does this Pull Request do?

  • Setup Python formatter that runs isort and black.
  • Rename scala formatter, so all three are consistent now.
  • Resolves #488
  • Normalize all the formatters
  • Sets all the formatters to run, but not apply changes. It would be helpful to do this, but we can't because we can't do
    that, because we can't push to a person's PR.
  • Apply isort/black
  • Make sure isort and black are consistent in how they are opinionated
    about imports

How should this be tested?

  • TravisCI
  • GitHub Actions should all pass hopefully 🤞
- Setup Python formatter that runs isort and black.
- Rename scala formatter, so all three are consistent now.
- Resolves #488
- Normalize all the formatters
- Sets all the formatters to run, but not apply changes. It would be helpful to do this, but we can't because we can't do
that, because we can't push to a person's PR.
- Apply isort/black
- Make sure isort and black are consistent in how they are opinionated
about imports
@ruebot ruebot requested a review from ianmilligan1 Jun 26, 2020
ruebot added 2 commits Jun 26, 2020
@codecov
Copy link

codecov bot commented Jun 26, 2020

Codecov Report

Merging #489 into master will increase coverage by 0.65%.
The diff coverage is 93.17%.

@@             Coverage Diff              @@
##             master     #489      +/-   ##
============================================
+ Coverage     88.20%   88.85%   +0.65%     
  Complexity       57       57              
============================================
  Files            43       43              
  Lines           958     1014      +56     
  Branches         86       86              
============================================
+ Hits            845      901      +56     
  Misses           74       74              
  Partials         39       39              
@ianmilligan1 ianmilligan1 merged commit 00e8166 into master Jun 26, 2020
6 checks passed
6 checks passed
formatting
Details
formatting
Details
scalafmt-lint
Details
codecov/patch 93.17% of diff hit (target 88.20%)
Details
codecov/project 88.85% (+0.65%) compared to 0a7a90f
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ianmilligan1 ianmilligan1 deleted the issue-488 branch Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

2 participants
You can’t perform that action at this time.