Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Sign up
Appearance settings

Commit f69f73f

Browse files
feat(conventional_commits): allow exclamation in title on BC
Add a configuration (disable by default) to allow exclamation in commit title to ease identify BC simple git log parsing. Signed-off-by: Sebastien Fusilier <sebastien.fusilier@gmail.com>
1 parent c700d66 commit f69f73f

File tree

6 files changed

+77
-0
lines changed

6 files changed

+77
-0
lines changed

‎commitizen/cz/conventional_commits/conventional_commits.py‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,11 +154,21 @@ def message(self, answers: ConventionalCommitsAnswers) -> str: # type: ignore[o
154154
footer = answers["footer"]
155155
is_breaking_change = answers["is_breaking_change"]
156156

157+
# Check if breaking change exclamation in title is enabled
158+
breaking_change_exclamation_in_title = self.config.settings.get(
159+
"breaking_change_exclamation_in_title", False
160+
)
161+
157162
if scope:
158163
scope = f"({scope})"
159164
if body:
160165
body = f"\n\n{body}"
161166
if is_breaking_change:
167+
if breaking_change_exclamation_in_title:
168+
if scope:
169+
scope = f"{scope}!"
170+
else:
171+
prefix = f"{prefix}!"
162172
footer = f"BREAKING CHANGE: {footer}"
163173
if footer:
164174
footer = f"\n\n{footer}"

‎commitizen/defaults.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ class Settings(TypedDict, total=False):
6161
version_scheme: str | None
6262
version_type: str | None
6363
version: str | None
64+
breaking_change_exclamation_in_title: bool
6465

6566

6667
CONFIG_FILES: list[str] = [
@@ -108,6 +109,7 @@ class Settings(TypedDict, total=False):
108109
"always_signoff": False,
109110
"template": None, # default provided by plugin
110111
"extras": {},
112+
"breaking_change_exclamation_in_title": False,
111113
}
112114

113115
MAJOR = "MAJOR"

‎docs/config.md‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,15 @@ Use annotated tags instead of lightweight tags. [See difference][annotated-tags-
9696

9797
Create custom commit message. Useful to skip CI. [Read more][bump_message]
9898

99+
### `breaking_change_exclamation_in_title`
100+
101+
Type: `bool`
102+
103+
Default: `false`
104+
105+
When true, breaking changes will be indicated by an exclamation mark in the commit title (e.g., `feat!: breaking change`).
106+
When false, breaking changes will be indicated by `BREAKING CHANGE:` in the footer. [Read more][breaking-change-exclamation]
107+
99108
### `retry_after_failure`
100109

101110
- Type: `bool`

‎docs/tutorials/writing_commits.md‎

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,10 @@ add to your commit body the following `BREAKING CHANGE`.
1313
Using these three keywords will allow the proper identification of the semantic version.
1414
Of course, there are other keywords, but I'll leave it to the reader to explore them.
1515

16+
Note: You can also indicate breaking changes by adding an exclamation mark in the commit title
17+
(e.g., `feat!: breaking change`) by setting the `breaking_change_exclamation_in_title`
18+
configuration option to `true`. [Read more][breaking-change-config]
19+
1620
## Writing commits
1721

1822
Now to the important part: when writing commits, it's important to think about:
@@ -44,3 +48,4 @@ Emojis may be added as well (e.g., see [cz-emoji][cz_emoji]), which requires the
4448
[conventional_commits]: https://www.conventionalcommits.org
4549
[cz_emoji]: https://commitizen-tools.github.io/commitizen/third-party-commitizen/#cz-emoji
4650
[configuration]: ../config.md#encoding
51+
[breaking-change-config]: ../config.md#breaking_change_exclamation_in_title

‎tests/test_conf.py‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -95,6 +95,7 @@
9595
"always_signoff": False,
9696
"template": None,
9797
"extras": {},
98+
"breaking_change_exclamation_in_title": False,
9899
}
99100

100101
_new_settings: dict[str, Any] = {
@@ -126,6 +127,7 @@
126127
"always_signoff": False,
127128
"template": None,
128129
"extras": {},
130+
"breaking_change_exclamation_in_title": False,
129131
}
130132

131133

‎tests/test_cz_conventional_commits.py‎

Lines changed: 49 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,6 +105,55 @@ def test_breaking_change_in_footer(config):
105105
)
106106

107107

108+
@pytest.mark.parametrize(
109+
"scope,breaking_change_exclamation_in_title,expected_message",
110+
[
111+
# Test with scope and breaking_change_exclamation_in_title enabled
112+
(
113+
"users",
114+
True,
115+
"feat(users)!: email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users",
116+
),
117+
# Test without scope and breaking_change_exclamation_in_title enabled
118+
(
119+
"",
120+
True,
121+
"feat!: email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users",
122+
),
123+
# Test with scope and breaking_change_exclamation_in_title disabled
124+
(
125+
"users",
126+
False,
127+
"feat(users): email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users",
128+
),
129+
# Test without scope and breaking_change_exclamation_in_title disabled
130+
(
131+
"",
132+
False,
133+
"feat: email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users",
134+
),
135+
],
136+
)
137+
def test_breaking_change_message_formats(
138+
config, scope, breaking_change_exclamation_in_title, expected_message
139+
):
140+
# Set the breaking_change_exclamation_in_title setting
141+
config.settings["breaking_change_exclamation_in_title"] = (
142+
breaking_change_exclamation_in_title
143+
)
144+
conventional_commits = ConventionalCommitsCz(config)
145+
answers = {
146+
"prefix": "feat",
147+
"scope": scope,
148+
"subject": "email pattern corrected",
149+
"is_breaking_change": True,
150+
"body": "complete content",
151+
"footer": "migrate by renaming user to users",
152+
}
153+
message = conventional_commits.message(answers)
154+
assert message == expected_message
155+
156+
108157
def test_example(config):
109158
"""just testing a string is returned. not the content"""
110159
conventional_commits = ConventionalCommitsCz(config)

0 commit comments

Comments
(0)

AltStyle によって変換されたページ (->オリジナル) /