Skip to content

Test PR 423 'invalid prefix'#455

Closed
oguzkocer wants to merge 8 commits intotrunkfrom
test-pr-423/invalid-prefix
Closed

Test PR 423 'invalid prefix'#455
oguzkocer wants to merge 8 commits intotrunkfrom
test-pr-423/invalid-prefix

Conversation

@oguzkocer
Copy link
Copy Markdown
Contributor

Testing #423

jkmassel added 5 commits April 3, 2026 15:28
Add a CHANGELOG.md following the release-toolkit format with historical
entries back to v0.3.0. Each version is categorized into Breaking Changes,
New Features, Bug Fixes, and Internal Changes.

The release script now automatically manages the Trunk section during
releases, and the developer workflow docs mention updating the changelog
when merging user-facing changes.
Address review feedback — the comprehensive per-PR lists are redundant
with the auto-generated GitHub Release notes from #392. The CHANGELOG
now focuses exclusively on breaking changes, which is the one thing
GitHub Releases can't surface ahead of time via the Trunk staging area.
Ensures every PR has either a valid Conventional Commits prefix in the
title (which triggers automatic labeling via label-pr.yml) or a manually
applied [Type] label. This prevents PRs from falling through the cracks
in auto-generated release notes.
The label-pr workflow already applies the [Type] Breaking Change label
when it sees ! in the title — no need to warn about it separately.
The new PR validation check ensures every PR has either a Conventional
Commits title or a type label, so GitHub Releases will reliably capture
breaking changes going forward. A manual CHANGELOG is redundant.
pull_request_target only runs the workflow from the default branch, so
the check can't fire until it's merged — a chicken-and-egg problem.

This workflow only needs read access (title + labels), so pull_request
is the correct trigger. pull_request_target is only needed for the
label-pr workflow which writes labels.
@jkmassel jkmassel force-pushed the test-pr-423/invalid-prefix branch from 715aaf0 to 9faef71 Compare April 15, 2026 18:17
jkmassel and others added 2 commits April 15, 2026 12:18
Force-pushes (e.g. rebases) fire the synchronize event, not opened or
edited. Without this, the check wouldn't re-run after a rebase.
@jkmassel jkmassel force-pushed the test-pr-423/invalid-prefix branch from 9faef71 to 2679c4d Compare April 15, 2026 18:18
@jkmassel jkmassel force-pushed the jkmassel/add-changelog branch 2 times, most recently from f8e2aaa to aeb020d Compare April 15, 2026 20:28
Base automatically changed from jkmassel/add-changelog to trunk April 15, 2026 20:41
@oguzkocer oguzkocer closed this Apr 15, 2026
@oguzkocer oguzkocer deleted the test-pr-423/invalid-prefix branch April 15, 2026 20: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.

2 participants