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

Use the pre-commit package with flake8 and black #483

Merged
merged 9 commits into from
Apr 11, 2020
Merged

Conversation

mwouts
Copy link
Owner

@mwouts mwouts commented Apr 5, 2020

Todo:

Closes #482

@codecov
Copy link

codecov bot commented Apr 5, 2020

Codecov Report

Merging #483 into master will increase coverage by 0.00%.
The diff coverage is 99.02%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #483   +/-   ##
=======================================
  Coverage   98.75%   98.76%           
=======================================
  Files          81       81           
  Lines        8060     8067    +7     
=======================================
+ Hits         7960     7967    +7     
  Misses        100      100           
Impacted Files Coverage Δ
tests/test_read_simple_rmd.py 100.00% <ø> (ø)
tests/test_read_simple_rust.py 100.00% <ø> (ø)
tests/test_read_simple_scheme.py 100.00% <ø> (ø)
tests/test_read_simple_sphinx.py 100.00% <ø> (ø)
tests/test_read_write_functions.py 100.00% <ø> (ø)
tests/test_rmd_to_ipynb.py 100.00% <ø> (ø)
tests/test_save_multiple.py 100.00% <ø> (ø)
tests/test_stringparser.py 100.00% <ø> (ø)
tests/test_trust_notebook.py 100.00% <ø> (ø)
tests/test_unicode.py 100.00% <ø> (ø)
... and 80 more

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 a0164dc...edbb1c6. Read the comment docs.

@mwouts mwouts added this to the 1.5.0 milestone Apr 8, 2020
@mwouts mwouts force-pushed the pre_commit_black branch from d2c59d5 to edbb1c6 Compare April 11, 2020 22:42
@mwouts mwouts merged commit f86f55f into master Apr 11, 2020
@mwouts mwouts deleted the pre_commit_black branch April 11, 2020 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Activate pre-commit reformatting and checks for the Jupytext repository
1 participant