Skip to content

Also check commit title for changelog entry#5968

Merged
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/changelog-checking-fix
Feb 17, 2022
Merged

Also check commit title for changelog entry#5968
mitchellhenke merged 1 commit intomainfrom
mitchellhenke/changelog-checking-fix

Conversation

@mitchellhenke
Copy link
Contributor

#5945 ran into some issues where the changelog was in the commit "title" rather than the commit message

This PR changes it so we check the commit messages as before, but then try to fall back to the commit title

Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

changelog: Internal, Continuous Integration, Write Changelog entries in commit messages
@mitchellhenke mitchellhenke force-pushed the mitchellhenke/changelog-checking-fix branch from b2069f6 to 36822c3 Compare February 17, 2022 19:10
@mitchellhenke mitchellhenke merged commit c3b912a into main Feb 17, 2022
@mitchellhenke mitchellhenke deleted the mitchellhenke/changelog-checking-fix branch February 17, 2022 19:23
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