Conversation
Co-authored-by: weinong <4204090+weinong@users.noreply.github.com>
|
@copilot I think this PR title will make this change included in the changelog generator. |
Co-authored-by: weinong <4204090+weinong@users.noreply.github.com>
Good catch! Fixed in e6c6cd8 — updated the changelog generator to also exclude PR titles matching |
|
@copilot would it be more reliable to use release label on the PR? |
Co-authored-by: weinong <4204090+weinong@users.noreply.github.com>
Done in f337e8c. Replaced the fragile title-regex approach with a |
Adds the v0.2.15 changelog entry covering PRs merged since v0.2.14, and fixes the changelog generator to exclude release-related PRs from future changelogs.
Changes
cache.Replacewhen running non-root ([Bug Fix] - PoP token flow crash with nil pointer in cache.Replace when running non-root #736, @vineeth-thumma)Changelog Generator Fix
Updated
hack/changelog-generator/main.goto exclude PRs carrying areleaselabel from the generated changelog. This is more reliable than title-pattern matching. A newhasLabelhelper (case-insensitive) was added, andgetMergedPRsSincewas updated to filter out any PR with thereleaselabel in addition to the existing version-prefixed title filter. AddedTestHasLabelinmain_test.goto cover the new helper.Adding the
releaselabel to changelog update PRs (like this one) will prevent them from appearing in future changelogs.✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.