Skip to content

fix: prefer git config user.name for author validation in BaseValidator - #461

Merged
shenxianpeng merged 3 commits into
mainfrom
bugfix/check-ai-commit-messaage
Jul 12, 2026
Merged

fix: prefer git config user.name for author validation in BaseValidator#461
shenxianpeng merged 3 commits into
mainfrom
bugfix/check-ai-commit-messaage

fix: skip unnecessary subprocess call in _should_skip_branch_validation

1f3ec2a
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 12, 2026 in 31s

Annotations

Check failure on line 418 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "dependabot[bot]" 7 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl9y&open=AZ85VNtzv4VYgpKmXl9y&pullRequest=461

Check failure on line 397 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "commit_check.engine.get_commit_info" 21 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl93&open=AZ85VNtzv4VYgpKmXl93&pullRequest=461

Check failure on line 422 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "commit_check.engine.has_commits" 4 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl94&open=AZ85VNtzv4VYgpKmXl94&pullRequest=461

Check failure on line 268 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal r"^feature/" 4 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl95&open=AZ85VNtzv4VYgpKmXl95&pullRequest=461

Check failure on line 424 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "commit_check.engine.get_branch_name" 3 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl9z&open=AZ85VNtzv4VYgpKmXl9z&pullRequest=461

Check failure on line 401 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "Alice Developer" 4 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl90&open=AZ85VNtzv4VYgpKmXl90&pullRequest=461

Check failure on line 392 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "pre-commit-ci[bot]" 4 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl92&open=AZ85VNtzv4VYgpKmXl92&pullRequest=461

Check failure on line 29 in tests/engine_test.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "commit_check.engine.get_git_config_value" 7 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNtzv4VYgpKmXl91&open=AZ85VNtzv4VYgpKmXl91&pullRequest=461

Check failure on line 121 in commit_check/engine.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Define a constant instead of duplicating this literal "user.name" 3 times.

See more on https://sonarcloud.io/project/issues?id=commit-check_commit-check&issues=AZ85VNwbv4VYgpKmXl96&open=AZ85VNwbv4VYgpKmXl96&pullRequest=461