feat: Add nbstripout

[robinholzi]

Motivation

As we have a couple of Jupyter notebooks in the repo now, let's ensure nobody commits the notebooks without clearing the outputs.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.02%. Comparing base (0ec6b7c) to head (3825a5a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #579   +/-   ##
=======================================
  Coverage   83.02%   83.02%           
=======================================
  Files         226      226           
  Lines       10532    10532           
=======================================
  Hits         8744     8744           
  Misses       1788     1788           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[github-actions]

^{( % to main)
( % to main)}

[MaxiBoether]

Generally LGTM, there seems to be a mypy error from a new mypy version. I wonder whether nbstripout can also be part of the CI? I guess it's better to also enforce this in the CI rather than just use pre commit hooks.

[robinholzi]

Generally LGTM, there seems to be a mypy error from a new mypy version. I wonder whether nbstripout can also be part of the CI? I guess it's better to also enforce this in the CI rather than just use pre commit hooks.

Yep, we can easily run the pre-commit hooks in the CI :) I wanted to wait with this until we enable ruff on the whole code base. Do you intend to merge #487? If yes, I'd rather wait until this is done to avoid conflicts.