Feature request: commitizen should support revert commits
#142
Comments
|
I add that IMHO, other than Edit: after some deeper research, I now agree that |
|
I now noticed the following definition of the def schema_pattern(self) -> str:
PATTERN = (
r"(build|ci|docs|feat|fix|perf|refactor|style|test|chore|revert|bump)"
r"(\(\S+\))?:\s.*"
)
return PATTERNSo it includes both |
|
To have |
|
As for command like |
I wouldn't add them, those 2 are added to the "pattern" in order to prevent backward errors, but officially they are not part of conventional commits, we are using the latest types from Angular (they used to but were removed).Using Regarding the |
|
When checking the commit message through |
|
This issue is raised more than one time (I had the same question when I first use commizen as well.) Maybe we could consider adding it to FAQ in documentation? |
|
OK, in this case I would document it in |
|
Thanks for the feedback |
I know that a similar request was discussed before, however
angulardoes contemplate revert commits.As a minimum,
cz checkshould accept commit messages beginning withrevert:, otherwise when it is configured as apre-commithook it does not allow revert commits.Moreover, IMHO a
cz revertcommand should be added, accepting the<commit>to revert. It should rungit revert <commit>, including in the message body:This reverts commit <hash>.as perangularinstructions. See information ongit revert.For examples of revert commit messages, see for instance
standard-versionandconventional-changelogcommit logs.Notice that both
commitlintandgitlintsupport revert commits.The text was updated successfully, but these errors were encountered: