Skip to content

Add pre-commit nbstripout for notebook cleanup - #45

Merged
lawrennd merged 1 commit into
mainfrom
chore/pre-commit-nbstripout
Nov 13, 2026
Merged

lawrennd merged 1 commit into
mainfrom
chore/pre-commit-nbstripout

Conversation

@lawrennd

Copy link
Copy Markdown
Member

Summary

  • Add .pre-commit-config.yaml with nbstripout 0.8.2 so Jupyter outputs / execution counts are stripped before commit.
  • Document pre-commit install in the README.
  • Normalize existing notebooks under the hook (damianou2013 Colab notebooks + examples/tutorial.ipynb).

Test plan

  • pre-commit run nbstripout --all-files (second run clean)
  • pip install pre-commit && pre-commit install then commit a notebook with outputs and confirm they are stripped

Strip outputs and execution counts on commit; document install in README
and normalize existing notebooks (including tutorial.ipynb).
@codecov

codecov Bot commented Nov 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.96%. Comparing base (8c87f6b) to head (6e5ac29).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage   74.96%   74.96%           
=======================================
  Files          17       17           
  Lines        1254     1254           
  Branches      185      185           
=======================================
  Hits          940      940           
  Misses        255      255           
  Partials       59       59           
Flag Coverage Δ
unittests 74.96% <ø> (ø)

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lawrennd
lawrennd merged commit 5596423 into main Nov 13, 2026
6 checks passed
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.

1 participant