Skip to content

chore: allow revert commit/PR - #62

Merged
ltshb merged 1 commit into
masterfrom
chore/allow-revert
Nov 4, 2026
Merged

ltshb merged 1 commit into
masterfrom
chore/allow-revert

Conversation

@ltshb

@ltshb ltshb commented Nov 4, 2026

Copy link
Copy Markdown
Contributor

Sometimes we use the github revert button on a PR that automatically open a PR with a revert commit. This is quite useful, but unfortunately we cannot configure github to use a special git commit message formatting and/or PR title.

So here we allow PR title and git commit message that starts with Revert "

Sometimes we use the github revert button on a PR that automatically open
a PR with a revert commit. This is quite useful, but unfortunately we cannot
configure github to use a special git commit message formatting and/or PR title.

So here we allow PR title and git commit message that starts with `Revert "`
@github-actions github-actions Bot added the skip-release-note Issue or PR excluded from the release notes label Nov 4, 2026
@ltshb
ltshb enabled auto-merge November 4, 2026 13:16
const jiraProjectKeys = JSON.parse(process.env.JIRA_PROJECT_KEYS);
const titlePattern = new RegExp(
`^(((${jiraProjectKeys.join("|")})-\\d+)+|(chore|fix|feat|revert|docs)(\\([^()]+\\))?): .+$`
`^(((${jiraProjectKeys.join("|")})-\\d+)+|(chore|fix|feat|revert|docs|Revert ")(\\([^()]+\\))?): .+$`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why not [Rr]evert

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Because automatic github revert commit and PR have Revert "blablabla" 😉

@ltshb
ltshb merged commit 5b23d68 into master Nov 4, 2026
5 checks passed
@ltshb
ltshb deleted the chore/allow-revert branch November 4, 2026 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-release-note Issue or PR excluded from the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants