Skip to content
This repository was archived by the owner on Oct 11, 2021. It is now read-only.

ci: add nbqa-flake8 pre-commit hook#517

Merged
redeboer merged 2 commits intomasterfrom
nbqa-flake8
Mar 12, 2021
Merged

ci: add nbqa-flake8 pre-commit hook#517
redeboer merged 2 commits intomasterfrom
nbqa-flake8

Conversation

@redeboer
Copy link
Copy Markdown
Member

@redeboer redeboer commented Mar 12, 2021

Run flake8 in notebooks in order to clean up code cells. The identified errors are fixed in this PR as well.

See pre-commit hooks and nbqa.rtfd.io.

@redeboer redeboer added 📝 Docs Improvements or additions to documentation 🖱️ DX Improvements to the Developer Experience labels Mar 12, 2021
@redeboer redeboer self-assigned this Mar 12, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2021

Codecov Report

Merging #517 (9407e8e) into master (f552473) will decrease coverage by 0.05%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
- Coverage   85.95%   85.90%   -0.06%     
==========================================
  Files          26       26              
  Lines        3518     3518              
  Branches      843      843              
==========================================
- Hits         3024     3022       -2     
- Misses        321      322       +1     
- Partials      173      174       +1     
Flag Coverage Δ
unittests 85.90% <ø> (-0.06%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/expertsystem/amplitude/helicity.py 85.53% <0.00%> (-0.63%) ⬇️

@redeboer redeboer merged commit 08aaf16 into master Mar 12, 2021
@redeboer redeboer deleted the nbqa-flake8 branch March 12, 2021 19:17
@redeboer redeboer added this to the Release 0.7.0 milestone Mar 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

📝 Docs Improvements or additions to documentation 🖱️ DX Improvements to the Developer Experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant