Skip to content

ci: Use gitcliff instead of release-please#1150

Merged
cedrik-fuoco-adsk merged 3 commits intoAcademySoftwareFoundation:mainfrom
cedrik-fuoco-adsk:use-gitcliff
Feb 26, 2026
Merged

ci: Use gitcliff instead of release-please#1150
cedrik-fuoco-adsk merged 3 commits intoAcademySoftwareFoundation:mainfrom
cedrik-fuoco-adsk:use-gitcliff

Conversation

@cedrik-fuoco-adsk
Copy link
Contributor

Use gitcliff instead of release-please

Linked issues

none

Summarize your change.

Replace release-please with git-cliff for automated changelog generation and release PR management.

This migration addresses release-please inability to parse commit messages containing parentheses in the body (e.g., multi-line markdown links), which caused commits to be silently dropped from changelogs.

I found the issue because the last commit description from @bernie-laberge broke release-please.

Note: I might have to do a subsequent PR if it does not detect the current version of OpenRV correctly. I wasn't able to test that in my fork.

Describe the reason for the change.

release-please uses @conventional-commits/parser, a strict grammar parser that attempts to parse the full commit message including the body and footers.

This parser fails on parentheses in commit bodies (see release-please#2564). git-cliff uses regex-based parsing on just the subject line, so commit bodies can contain any content without breaking.### Describe what you have tested and on which operating system.

Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
Signed-off-by: Cédrik Fuoco <cedrik.fuoco@autodesk.com>
@cedrik-fuoco-adsk
Copy link
Contributor Author

Sorry for the noise, it is ready to review @eloisebrosseau @bernie-laberge

Copy link
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

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

LGTM

@cedrik-fuoco-adsk cedrik-fuoco-adsk changed the title ci: use gitcliff instead of release-please ci: Use gitcliff instead of release-please Feb 26, 2026
@cedrik-fuoco-adsk cedrik-fuoco-adsk merged commit 1a414f3 into AcademySoftwareFoundation:main Feb 26, 2026
10 of 21 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.

3 participants