Skip to content

Tests: Add pre-commit link checking hook for changed files.#160

Open
brycelelbach wants to merge 1 commit intomainfrom
feature/pre-commit-link-checking
Open

Tests: Add pre-commit link checking hook for changed files.#160
brycelelbach wants to merge 1 commit intomainfrom
feature/pre-commit-link-checking

Conversation

@brycelelbach
Copy link
Collaborator

Summary

  • Add a test-links pre-commit hook that checks links in only the staged .md and .ipynb files, rather than the entire repo. Pre-commit passes the changed filenames directly to brev/test-links.bash.
  • Refactor brev/test-links.bash to accept multiple file and directory arguments. Notebooks are converted individually; jupyter is only required when notebooks are among the targets.
  • Rename the existing whole-repo link check to test-links-all at the manual stage (pre-commit run --hook-stage manual test-links-all).
  • Document pre-commit hook setup and the hook table in CONTRIBUTING.md.

@brycelelbach brycelelbach enabled auto-merge (rebase) March 11, 2026 18:54
@brycelelbach brycelelbach force-pushed the feature/pre-commit-link-checking branch from 0405042 to d297284 Compare March 11, 2026 21:14
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.

2 participants