Skip to content

Conversation

@nMoncho
Copy link

@nMoncho nMoncho commented Nov 20, 2025

As mentioned on https://www.conventionalcommits.org/en/v1.0.0/ other commit types are allowed. For example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:.

Currently the changelog.mustache template would drop commits that aren't fix|feat|breaking|chore, and we would probably like to see them in the Other Changes section.


{{#commits}}
{{#ifCommitType . type='^$'}}
{{#ifCommitType . type='^($^|(?!fix|feat|breaking|chore).*)'}}
Copy link
Author

Choose a reason for hiding this comment

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

#ifCommitType . type='^$' expects the commit type to be empty. By using the same expression as line 50, we can include both empty types, and other types such as refactor:, docs:, etc.

Copy link
Owner

Choose a reason for hiding this comment

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

I would encourage all users to create their own custom template and adjust it exactly as they want. All tools using this library should offer that feature.


[{{subString hash 0 5}}](https://github.com/{{ownerName}}/{{repoName}}/commit/{{hash}}) {{authorName}} *{{commitTime}}*

{{/ifCommitTypeOtherThan}}
Copy link
Author

Choose a reason for hiding this comment

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

We also need to make sure the chores that aren't dependency updates are included in the list, as the chore(deps) were included in Dependency updates section

Copy link
Owner

Choose a reason for hiding this comment

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

This probably means you should have your own complete template. I personally dont want the chores in the changelog.

@nMoncho
Copy link
Author

nMoncho commented Nov 21, 2025

Fair enough, thank you for checking this so quickly. I'll close the PR.

@nMoncho nMoncho closed this Nov 21, 2025
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