Skip to content

fix: skip PR title check on push#88

Merged
longlho merged 1 commit into
mainfrom
codex/fix-main-hook-title
May 30, 2026
Merged

fix: skip PR title check on push#88
longlho merged 1 commit into
mainfrom
codex/fix-main-hook-title

Conversation

@longlho
Copy link
Copy Markdown
Contributor

@longlho longlho commented May 30, 2026

Summary

  • treat empty PR title env vars as absent on non-PR events
  • keep empty PR titles failing on pull_request events

Test plan

  • PR_TITLE= GITHUB_EVENT_NAME=push pnpm lefthook run pre-commit --force
  • PR_TITLE="ci: add lefthook workflow" GITHUB_EVENT_NAME=pull_request pnpm lefthook run pre-commit --force
  • PR_TITLE= GITHUB_EVENT_NAME=pull_request node --experimental-transform-types tools/verify-pr-title.ts
  • pnpm lefthook validate
  • git diff --check

@longlho longlho merged commit 6830ca0 into main May 30, 2026
2 checks passed
@longlho longlho deleted the codex/fix-main-hook-title branch May 30, 2026 02:22
@pplx-oss pplx-oss mentioned this pull request May 30, 2026
Merged
longlho pushed a commit that referenced this pull request May 30, 2026
@pplx-oss
chore(main): release codescythe_cli 0.6.1 (#89)
a0a07c9 
🤖 I have created a release *beep* *boop*
---


##
[0.6.1](codescythe_cli_v0.6.0...codescythe_cli_v0.6.1)
(2026-05-30)


### Bug Fixes

* skip PR title check on push
([#88](#88))
([6830ca0](6830ca0))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@longlho