Skip to content

Add nbstripout pre-commit hook to reduce noisy notebook diffs #3585

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.

Sign up for GitHub

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

Open
Prathamesh8989 wants to merge 5 commits into
base: master
Choose a base branch
from
+9 −6
Open

Add nbstripout pre-commit hook to reduce noisy notebook diffs #3585

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
28cabe7
Fix pre-commit config and add venv to gitignore
Prathamesh8989 Feb 24, 2026
2d2ca6e
Fix pre-commit config and add venv to gitignore
Prathamesh8989 Feb 24, 2026
1a63a8f
Revert .gitignore changes as requested
Prathamesh8989 Feb 25, 2026
618a613
Merge branch 'master' into fix-precommit-config
Prathamesh8989 Feb 25, 2026
6d976be
Add nbstripout and enable jupyter support for ruff and black
Prathamesh8989 Feb 26, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 9 additions & 6 deletions .pre-commit-config.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
exclude: "^conda.recipe|assets|docs"

Copy link
Contributor

@omkar-334 omkar-334 Feb 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove this extra line

repos:
- repo: https://github.com/kynan/nbstripout
rev: 0.7.1
hooks:
- id: nbstripout
types: [jupyter]

- repo: https://github.com/pre-commit/pre-commit-hooks
rev: v5.0.0
hooks:
Expand All @@ -17,15 +24,11 @@ repos:
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.11.13
hooks:
# Run the linter
- id: ruff-check
types_or: [python, pyi]
# TODO: Enable when black and ruff format converge
# Run the formatter
# - id: ruff-format
# types_or: [python, pyi]
types_or: [python, pyi, jupyter]

- repo: https://github.com/psf/black
rev: 24.10.0
hooks:
- id: black
Copy link
Contributor

@aaishwarymishra aaishwarymishra Feb 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- id: black
- id: black
types_or: [python, pyi, jupyter]

types_or: [python, pyi, jupyter]