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

ci: run flake8 black on pre commit failure #131

Merged
merged 1 commit into from
Nov 27, 2021

Conversation

onerandomusername
Copy link
Member

Always run flake8 and black on pre-commit failure, as long as pre-commit wasn't cancelled or skipped.

@onerandomusername onerandomusername added a: CI Continuous Integration and Continuous Deployment p: normal Normal Priority labels Nov 25, 2021
@onerandomusername onerandomusername added the skip changelog Skip the need for a changelog entry for a Pull Request label Nov 25, 2021
@codecov
Copy link

codecov bot commented Nov 25, 2021

Codecov Report

Merging #131 (4b07917) into main (d0d7d78) will not change coverage.
The diff coverage is n/a.

❗ Current head 4b07917 differs from pull request most recent head 763a243. Consider uploading reports for the commit 763a243 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main     #131   +/-   ##
=======================================
  Coverage   70.90%   70.90%           
=======================================
  Files          33       33           
  Lines        1619     1619           
  Branches      199      199           
=======================================
  Hits         1148     1148           
  Misses        439      439           
  Partials       32       32           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eec59a3...763a243. Read the comment docs.

@Shivansh-007
Copy link
Member

Waiting for the debug file to be removed...

@onerandomusername onerandomusername force-pushed the ci/run-flake8-black-on-pre-commit-failure branch 2 times, most recently from a30e751 to 58d64fd Compare November 27, 2021 02:52
@Shivansh-007 Shivansh-007 changed the title ci/run flake8 black on pre commit failure ci: run flake8 black on pre commit failure Nov 27, 2021
@onerandomusername onerandomusername force-pushed the ci/run-flake8-black-on-pre-commit-failure branch from 58d64fd to 763a243 Compare November 27, 2021 02:55
@onerandomusername onerandomusername merged commit 6ab7918 into main Nov 27, 2021
@onerandomusername onerandomusername deleted the ci/run-flake8-black-on-pre-commit-failure branch November 27, 2021 02:59
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 1509595463

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 78.582%

Totals Coverage Status
Change from base Build 1509593147: 0.0%
Covered Lines: 1629
Relevant Lines: 2073

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a: CI Continuous Integration and Continuous Deployment p: normal Normal Priority skip changelog Skip the need for a changelog entry for a Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants