Skip to content

ci: add lefthook PR title check#87

Merged
longlho merged 1 commit into
mainfrom
codex/lefthook-pr-title
May 30, 2026
Merged

ci: add lefthook PR title check#87
longlho merged 1 commit into
mainfrom
codex/lefthook-pr-title

Conversation

@longlho
Copy link
Copy Markdown
Contributor

@longlho longlho commented May 30, 2026

Summary

  • add lefthook with a pre-commit PR title validator
  • add a GitHub Actions workflow that runs lefthook on changed files
  • document the hook and PR title convention in AGENTS.md
  • ignore package-local node_modules output created by pnpm installs

Test plan

  • pnpm lefthook validate
  • PR_TITLE="ci: add lefthook workflow" pnpm lefthook run pre-commit --force
  • node --experimental-transform-types tools/verify-pr-title.ts --title "fix(codescythe_cli)!: handle config lookup"
  • git diff --check
  • PR checks: Verify hooks, Test Rust

@longlho longlho force-pushed the codex/lefthook-pr-title branch 2 times, most recently from f1e16b6 to 6d1098f Compare May 30, 2026 01:52
@longlho longlho force-pushed the codex/lefthook-pr-title branch from 6d1098f to 8fe6ec7 Compare May 30, 2026 01:53
@longlho longlho enabled auto-merge (squash) May 30, 2026 01:54
@longlho longlho merged commit 12e6a62 into main May 30, 2026
2 checks passed
@longlho longlho deleted the codex/lefthook-pr-title branch May 30, 2026 01:54
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