Skip to content

FIX Handle fix prefix with dash - #273

Merged
blueo merged 2 commits into
silverstripe:masterfrom
creative-commoners:pulls/master/fix-changelog-fix
Aug 28, 2026
Merged

blueo merged 2 commits into
silverstripe:masterfrom
creative-commoners:pulls/master/fix-changelog-fix

Conversation

@emteknetnz

@emteknetnz emteknetnz commented Aug 28, 2026 •

Copy link
Copy Markdown
Member

Issue https://github.com/silverstripeltd/product-issues/issues/934

2x fixes:

A)
There was a commit in the changelog that was FIX - lorem ipsum - it ended up getting the FIX chopped and displayed as a backtick escaped - lorem ipsum instead - it displayed in the 'Bugfixes' section.

Instead it should still show in the bugfixes section, just not have backticks and display as lorem ispum with no backticks

B)
Several commits e.g. silverstripe/silverstripe-template-engine@e4f5f4e weren't getting picked up in the changelog because they look to have been squash merged, but still retain the 'Merge pull request' line you get from a regular merge - possible github changed something at their end?

Fix it to fallback to commit message body

@emteknetnz
emteknetnz marked this pull request as ready for review August 28, 2026 01:53

@blueo blueo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

works as described. noting security ones have a different format eg [CVE-xzy] message but that can be done separately.
I'll attempt to use the proper message when merging this one haha 😛

@blueo
blueo merged commit 9530105 into silverstripe:master Aug 28, 2026
9 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