style(ruff): run ruff check --fix

gbaian10 · Lee-W · commit f60e9a228718 · 2025-06-08T16:55:35.000+09:00
Auto fix RUF022 and RUF100
diff --git a/commitizen/commands/__init__.py b/commitizen/commands/__init__.py
@@ -11,13 +11,13 @@
 
 __all__ = (
     "Bump",
+    "Changelog",
     "Check",
     "Commit",
-    "Changelog",
     "Example",
     "Info",
+    "Init",
     "ListCz",
     "Schema",
     "Version",
-    "Init",
 )
diff --git a/commitizen/commands/bump.py b/commitizen/commands/bump.py
@@ -135,7 +135,7 @@ def find_increment(self, commits: list[git.GitCommit]) -> Increment | None:
             )
         return bump.find_increment(commits, regex=bump_pattern, increments_map=bump_map)
 
-    def __call__(self) -> None:  # noqa: C901
+    def __call__(self) -> None:
         """Steps executed to bump."""
         provider = get_provider(self.config)
 
diff --git a/commitizen/cz/conventional_commits/conventional_commits.py b/commitizen/cz/conventional_commits/conventional_commits.py
@@ -31,7 +31,7 @@ class ConventionalCommitsCz(BaseCommitizen):
     bump_pattern = defaults.BUMP_PATTERN
     bump_map = defaults.BUMP_MAP
     bump_map_major_version_zero = defaults.BUMP_MAP_MAJOR_VERSION_ZERO
-    commit_parser = r"^((?P<change_type>feat|fix|refactor|perf|BREAKING CHANGE)(?:\((?P<scope>[^()\r\n]*)\)|\()?(?P<breaking>!)?|\w+!):\s(?P<message>.*)?"  # noqa
+    commit_parser = r"^((?P<change_type>feat|fix|refactor|perf|BREAKING CHANGE)(?:\((?P<scope>[^()\r\n]*)\)|\()?(?P<breaking>!)?|\w+!):\s(?P<message>.*)?"
     change_type_map = {
         "feat": "Feat",
         "fix": "Fix",
diff --git a/commitizen/cz/jira/jira.py b/commitizen/cz/jira/jira.py
@@ -67,7 +67,7 @@ def example(self) -> str:
         )
 
     def schema(self) -> str:
-        return "<ignored text> <ISSUE_KEY> <ignored text> #<COMMAND> <optional COMMAND_ARGUMENTS>"  # noqa
+        return "<ignored text> <ISSUE_KEY> <ignored text> #<COMMAND> <optional COMMAND_ARGUMENTS>"
 
     def schema_pattern(self) -> str:
         return r".*[A-Z]{2,}\-[0-9]+( #| .* #).+( #.+)*"
diff --git a/commitizen/providers/__init__.py b/commitizen/providers/__init__.py
@@ -21,7 +21,6 @@
 from commitizen.providers.uv_provider import UvProvider
 
 __all__ = [
-    "get_provider",
     "CargoProvider",
     "CommitizenProvider",
     "ComposerProvider",
@@ -30,6 +29,7 @@
     "PoetryProvider",
     "ScmProvider",
     "UvProvider",
+    "get_provider",
 ]
 
 PROVIDER_ENTRYPOINT = "commitizen.provider"
diff --git a/tests/conftest.py b/tests/conftest.py
@@ -169,7 +169,7 @@ class SemverCommitizen(BaseCommitizen):
         "patch": "PATCH",
     }
     changelog_pattern = r"^(patch|minor|major)"
-    commit_parser = r"^(?P<change_type>patch|minor|major)(?:\((?P<scope>[^()\r\n]*)\)|\()?:?\s(?P<message>.+)"  # noqa
+    commit_parser = r"^(?P<change_type>patch|minor|major)(?:\((?P<scope>[^()\r\n]*)\)|\()?:?\s(?P<message>.+)"
     change_type_map = {
         "major": "Breaking Changes",
         "minor": "Features",
diff --git a/tests/test_bump_find_increment.py b/tests/test_bump_find_increment.py
@@ -32,14 +32,14 @@
 MAJOR_INCREMENTS_BREAKING_CHANGE_CC = [
     "feat(cli): added version",
     "docs(README): motivation",
-    "BREAKING CHANGE: `extends` key in config file is now used for extending other config files",  # noqa
+    "BREAKING CHANGE: `extends` key in config file is now used for extending other config files",
     "fix(setup.py): future is now required for every python version",
 ]
 
 MAJOR_INCREMENTS_BREAKING_CHANGE_ALT_CC = [
     "feat(cli): added version",
     "docs(README): motivation",
-    "BREAKING-CHANGE: `extends` key in config file is now used for extending other config files",  # noqa
+    "BREAKING-CHANGE: `extends` key in config file is now used for extending other config files",
     "fix(setup.py): future is now required for every python version",
 ]
 
diff --git a/tests/test_cz_conventional_commits.py b/tests/test_cz_conventional_commits.py
@@ -89,7 +89,7 @@ def test_long_answer(config):
     message = conventional_commits.message(answers)
     assert (
         message
-        == "fix(users): email pattern corrected\n\ncomplete content\n\ncloses #24"  # noqa
+        == "fix(users): email pattern corrected\n\ncomplete content\n\ncloses #24"
     )
 
 
@@ -107,7 +107,7 @@ def test_breaking_change_in_footer(config):
     print(message)
     assert (
         message
-        == "fix(users): email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users"  # noqa
+        == "fix(users): email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users"
     )
 
 

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ def find_increment(self, commits: list[git.GitCommit]) -> Increment \| None:`
`135`	`135`	`)`
`136`	`136`	`return bump.find_increment(commits, regex=bump_pattern, increments_map=bump_map)`
`137`	`137`
`138`		`- def __call__(self) -> None: # noqa: C901`
	`138`	`+ def __call__(self) -> None:`
`139`	`139`	`"""Steps executed to bump."""`
`140`	`140`	`provider = get_provider(self.config)`
`141`	`141`
Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ def example(self) -> str:`
`67`	`67`	`)`
`68`	`68`
`69`	`69`	`def schema(self) -> str:`
`70`		`- return "<ignored text> <ISSUE_KEY> <ignored text> #<COMMAND> <optional COMMAND_ARGUMENTS>" # noqa`
	`70`	`+ return "<ignored text> <ISSUE_KEY> <ignored text> #<COMMAND> <optional COMMAND_ARGUMENTS>"`
`71`	`71`
`72`	`72`	`def schema_pattern(self) -> str:`
`73`	`73`	`return r".[A-Z]{2,}\-[0-9]+( #\| . #).+( #.+)*"`
Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ class SemverCommitizen(BaseCommitizen):`
`169`	`169`	`"patch": "PATCH",`
`170`	`170`	`}`
`171`	`171`	`changelog_pattern = r"^(patch\|minor\|major)"`
`172`		`- commit_parser = r"^(?P<change_type>patch\|minor\|major)(?:\((?P<scope>[^()\r\n]*)\)\|\()?:?\s(?P<message>.+)" # noqa`
	`172`	`+ commit_parser = r"^(?P<change_type>patch\|minor\|major)(?:\((?P<scope>[^()\r\n]*)\)\|\()?:?\s(?P<message>.+)"`
`173`	`173`	`change_type_map = {`
`174`	`174`	`"major": "Breaking Changes",`
`175`	`175`	`"minor": "Features",`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ def test_long_answer(config):`
`89`	`89`	`message = conventional_commits.message(answers)`
`90`	`90`	`assert (`
`91`	`91`	`message`
`92`		`- == "fix(users): email pattern corrected\n\ncomplete content\n\ncloses #24" # noqa`
	`92`	`+ == "fix(users): email pattern corrected\n\ncomplete content\n\ncloses #24"`
`93`	`93`	`)`
`94`	`94`
`95`	`95`
`@@ -107,7 +107,7 @@ def test_breaking_change_in_footer(config):`
`107`	`107`	`print(message)`
`108`	`108`	`assert (`
`109`	`109`	`message`
`110`		`- == "fix(users): email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users" # noqa`
	`110`	`+ == "fix(users): email pattern corrected\n\ncomplete content\n\nBREAKING CHANGE: migrate by renaming user to users"`
`111`	`111`	`)`
`112`	`112`
`113`	`113`