Skip to content

ci: keep non-conventional commits in the changelog#42

Merged
takaebato merged 1 commit into
masterfrom
fix/changelog-unconventional
Jun 28, 2026
Merged

ci: keep non-conventional commits in the changelog#42
takaebato merged 1 commit into
masterfrom
fix/changelog-unconventional

Conversation

@takaebato

Copy link
Copy Markdown
Owner

Problem

The first Release PR (#41) silently dropped every commit that isn't a Conventional Commit — including major work like #31 (operator extraction) and #33 (the extraction-correctness pass), plus the external contributions #7/#9.

Root cause: the release-plz version that ran (0.3.159) inherits git-cliff's default filter_unconventional = true, which discards non-conventional commits before the commit_parsers run — so the .* catch-all never sees them.

Fix

Set filter_unconventional = false in release-plz.toml. Non-conventional commits are kept and routed to Other Changes via the catch-all.

Effect

🤖 Generated with Claude Code

release-plz 0.3.159 inherits git-cliff filter_unconventional=true, which drops commits without a Conventional Commits prefix before the parsers run, so the catch-all never groups them. Setting filter_unconventional=false keeps the historical backlog (e.g. #31/#33) in "Other Changes".

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

✅ PR title follows the Conventional Commits spec.

@takaebato takaebato merged commit 6ef0361 into master Jun 28, 2026
8 checks passed
@takaebato takaebato deleted the fix/changelog-unconventional branch June 28, 2026 09:05
takaebato added a commit that referenced this pull request Jun 28, 2026
## Problem

PR #42 added `filter_unconventional = false` to `release-plz.toml`, but
that key is **not supported by release-plz 0.3.159** (the latest release
— it only exists in unreleased release-plz). Since #42 merged, the
release-plz workflow on `master` now fails to parse the config:

```
unknown field `filter_unconventional`, expected one of `header`, `body`, `trim`,
`commit_preprocessors`, `postprocessors`, `sort_commits`, `link_parsers`,
`commit_parsers`, `protect_breaking_commits`, `tag_pattern`
```

## Fix

Remove the unsupported key, restoring a valid config so release-plz
works again.

## Follow-up (separate)

The original goal — including pre-convention (non-`type:`) commits like
#31/#33 and the external #7/#9 in the changelog — has **no released
config/upgrade fix** (the toggle is unreleased). It is a one-time
historical-backlog gap; future commits are all conventional (PR-title
lint). We'll handle that gap manually rather than via config.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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