Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add new pre-commit entry#570

Merged
delucchi-cmu merged 2 commits intomainlincc-frameworks/python-project-template:mainfrom
issue/558/pre-commitlincc-frameworks/python-project-template:issue/558/pre-commitCopy head branch name to clipboard
Jan 26, 2026
Merged

Add new pre-commit entry#570
delucchi-cmu merged 2 commits intomainlincc-frameworks/python-project-template:mainfrom
issue/558/pre-commitlincc-frameworks/python-project-template:issue/558/pre-commitCopy head branch name to clipboard

Conversation

@delucchi-cmu
Copy link
Contributor

Change Description

Closes #558

This PR kind of expanded...

On the PPT itself:

On hydrated projects:

  • Adds pre-executed-nb-never-execute when you have docs and notebooks.

Checklist

  • This PR is meant for the lincc-frameworks/python-project-template repo and not a downstream one instead.
  • This change is linked to an open issue
  • This change includes integration testing, or is small enough to be covered by existing tests

@delucchi-cmu delucchi-cmu requested a review from hombit January 26, 2026 20:20
@delucchi-cmu
Copy link
Contributor Author

You can see this in action with astronomy-commons/lsdb#1226

@delucchi-cmu delucchi-cmu merged commit f83b6e9 into main Jan 26, 2026
7 checks passed
@delucchi-cmu delucchi-cmu deleted the issue/558/pre-commit branch January 26, 2026 20:36
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.

Add a pre-commit hook to check that pre-executed notebooks are marked so

2 participants

Comments

Close sidebar
Morty Proxy This is a proxified and sanitized view of the page, visit original site.