Skip to content

feat: test PR 423 'valid prefix'#454

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

feat: test PR 423 'valid prefix'#454
oguzkocer wants to merge 8 commits intotrunkfrom
test-pr-423/feat-test

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/feat-test branch from e07aebf to 1f66565 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/feat-test branch from 1f66565 to 94394f2 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/feat-test 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

[Type] Enhancement A suggestion for improvement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants