Skip to content

Move docs checks from git hooks to CI - #704

Merged
bryantgillespie merged 5 commits into
mainfrom
bry/rethink-githooks
Jun 2, 2026
Merged

bryantgillespie merged 5 commits into
mainfrom
bry/rethink-githooks

Conversation

@bryantgillespie

Copy link
Copy Markdown
Member

Changes

  • Moves stable ID, redirect, and script type checks from local git hooks into docs CI for docs PRs, and documents manual stable ID/redirect workflows.
  • Removes hook setup from postinstall, deletes the hook scripts, switches repo scripts from tsx to Node native TypeScript, and moves script-only packages to devDependencies.
  • Updates search indexing workflow paths/install behavior/concurrency, and improves stable ID plus redirect failure guidance.

Potential Risks

  • Redirect coverage now runs as a PR gate; direct pushes to main skip redirect diff because main would compare against itself.

Review Notes

  • Local checks passed: pnpm typecheck:scripts, pnpm stable-ids:check, pnpm redirects:check, pnpm api-ref:generate, pnpm test:search.

@bryantgillespie
bryantgillespie requested a review from a team as a code owner June 2, 2026 13:54
@vercel

vercel Bot commented Jun 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Jun 2, 2026 2:52pm

Request Review

bryantgillespie and others added 2 commits June 2, 2026 10:36
- Add parseArgs/cleanupAlias/aliasFromSlot tests
- Validate --branch arg, dedupe orphan filter with a Set
- Move PR-close cleanup into its own workflow with dispatch input
- Comment --ignore-scripts rationale on CI install steps
@bryantgillespie
bryantgillespie merged commit 2c069d5 into main Jun 2, 2026
1 of 2 checks passed
@bryantgillespie
bryantgillespie deleted the bry/rethink-githooks branch June 2, 2026 14:48

This branch was successfully deployed

1 active deployment
Preview — a365846c Deployed Jun 2, 2026 by vercel[bot]
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