ci: add gitlint to pre-commit hook

JIRA: PSDK-216
risk: low

Author: hkad98

.gitlint

@@ -0,0 +1,19 @@
+# (C) 2025 GoodData Corporation
+[general]
+# do not enforce maximum line length rules
+ignore=T1, B1, B2, B5, T5
+
+# prevent a warning about regexes in the ignore-by-title section
+regex-style-search=true
+
+# enable our extended conventional commits rule
+extra-path=packages/repo-tools/src/quiver_monorepo/conventional_commit_gitlint_rule.py
+
+# ignore all release commits (merge and revert commits are ignored by default)
+[ignore-by-title]
+regex=^Release(.*)
+ignore=all
+
+# add the most likely scopes for each package
+[gdc-title-conventional-commits]
+scopes=gooddata-api-client,gooddata-dbt,gooddata-fdw,gooddata-flexconnect,gooddata-flight-server,gooddata-pandas,gooddata-sdk

.pre-commit-config.yaml

@@ -30,3 +30,7 @@ repos:
         args: [ "--update", "FILES" ]
         language: script
         types: [ text ]
+  - repo: https://github.com/jorisroovers/gitlint
+    rev: v0.19.1
+    hooks:
+      - id: gitlint

dev-requirements.txt

@@ -1,6 +1,8 @@
 -r ./tox-requirements.txt
 -r ./release-requirements.txt
+
 pre-commit~=4.0.1
+gitlint~=0.19.1
 
 -r ./fmt-requirements.txt
 

scripts/conventional_commit_gitlint_rule.py

@@ -0,0 +1,67 @@
+# (C) 2025 GoodData Corporation
+"""
+An extension of the CT1 rule from gitlint to enforce the conventional commit format.
+This version also allows specifying allowed scope values.
+"""
+
+import re
+
+from gitlint.options import ListOption
+from gitlint.rules import CommitMessageTitle, LineRule, RuleViolation
+
+RULE_REGEX = re.compile(r"([^(]+?)(?:\(([^)]+?)\))?!?: .+")
+
+DEFAULT_TYPES = [
+    "fix",
+    "feat",
+    "chore",
+    "docs",
+    "style",
+    "refactor",
+    "perf",
+    "test",
+    "revert",
+    "ci",
+    "build",
+]
+DEFAULT_SCOPES = []
+
+
+class ConventionalCommit(LineRule):
+    """This rule enforces the spec at https://www.conventionalcommits.org/."""
+
+    name = "gdc-title-conventional-commits"
+    id = "GD1"
+    target = CommitMessageTitle
+
+    options_spec = [
+        ListOption(
+            "types",
+            DEFAULT_TYPES,
+            "Comma separated list of allowed commit types.",
+        ),
+        ListOption(
+            "scopes",
+            DEFAULT_SCOPES,
+            "Comma separated list of allowed commit scopes.",
+        ),
+    ]
+
+    def validate(self, line, _commit):
+        violations = []
+        match = RULE_REGEX.match(line)
+
+        if not match:
+            msg = "Title does not follow ConventionalCommits.org format 'type(optional-scope): description'"
+            violations.append(RuleViolation(self.id, msg, line))
+        else:
+            line_commit_type = match.group(1)
+            line_commit_scope = match.group(2)
+            if line_commit_type not in self.options["types"].value:
+                opt_str = ", ".join(self.options["types"].value)
+                violations.append(RuleViolation(self.id, f"Title does not start with one of {opt_str}", line))
+            if line_commit_scope and line_commit_scope not in self.options["scopes"].value:
+                opt_str = ", ".join(self.options["scopes"].value)
+                violations.append(RuleViolation(self.id, f"Scope is defined and is not one of {opt_str}", line))
+
+        return violations