Skip to content

tests: fix the PR title check broken by the action bump to v6.1.1 - #410

Merged
zhuizhuhaomeng merged 1 commit into
openresty:masterfrom
tzssangglass:fix-pr-title-check
Sep 7, 2026
Merged

tests: fix the PR title check broken by the action bump to v6.1.1#410
zhuizhuhaomeng merged 1 commit into
openresty:masterfrom
tzssangglass:fix-pr-title-check

Conversation

@tzssangglass

@tzssangglass tzssangglass commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Since v6.1.0 each line of the types input is treated as a regex pattern auto-wrapped in ^ $ (amannn/action-semantic-pull-request#292), so the trailing "# comment" became part of the pattern: the line "bugfix # bug fixes" means regex ^bugfix # bug fixes$, which can never match the type token in a PR title. Every PR has failed the "Lint PR" check since the bump in db281a3.

Under @v4 the input was split on whitespace into plain strings, so "bugfix # bug fixes" accidentally yielded the bare token "bugfix", which matched. That accidental tolerance is gone in v6.

Fix: list each type as one bare line, as the v6 docs describe and as other repos pinning the same v6.1.1 SHA do:
https://github.com/projen/projen/blob/main/.github/workflows/pull-request-lint.yml#L23-L30

Since v6.1.0 each line of the `types` input is treated as a regex
pattern auto-wrapped in `^ $` (amannn/action-semantic-pull-request#292),
so the trailing "# comment" became part of the pattern: the line
"bugfix # bug fixes" means regex ^bugfix # bug fixes$, which can never
match the type token in a PR title. Every PR has failed the "Lint PR"
check since the bump in db281a3.

Under @v4 the input was split on whitespace into plain strings, so
"bugfix # bug fixes" accidentally yielded the bare token "bugfix", which
matched. That accidental tolerance is gone in v6.

Fix: list each type as one bare line on the strip block scalar (`|-`),
as the v6 docs describe and as other repos pinning the same v6.1.1 SHA
do:
https://github.com/projen/projen/blob/main/.github/workflows/pull-request-lint.yml#L23-L30
@zhuizhuhaomeng
zhuizhuhaomeng merged commit ebdafbe into openresty:master Sep 7, 2026
1 of 3 checks passed
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