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

Scope the nbqa analysis to the notebooks folder #41

Merged
merged 1 commit into from
Oct 17, 2020

Conversation

neomatrix369
Copy link
Owner

@neomatrix369 neomatrix369 commented Oct 17, 2020

Goal or purpose of the PR

Amending the config for nbqa to only run analysis of files in the notebooks folder, on the back of the discussion #40 (comment) (in #40)

Changes implemented in the PR

.pre-commit-config.yaml modified using files directive

Thanks @MarcoGorelli for the suggestion

Amending the config for `nbqa` to only run analysis of files in the `notebooks` folder
@codecov
Copy link

codecov bot commented Oct 17, 2020

Codecov Report

Merging #41 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           21        21           
  Lines          358       358           
  Branches        51        51           
=========================================
  Hits           358       358           

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 0bbaeb8...11ae1f5. Read the comment docs.

@neomatrix369 neomatrix369 merged commit 8d4d40c into master Oct 17, 2020
NLP Profiler automation moved this from Doing to Done Oct 17, 2020
@neomatrix369 neomatrix369 deleted the nbqa-config-scoping-scanning branch October 17, 2020 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
NLP Profiler
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

1 participant