Skip to content

ci: Add GitHub Action to validate commit message style #2071

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 9, 2025

Conversation

szymon-czapracki
Copy link
Contributor

This workflow checks all commits between the current branch and upstream/master. It enforces subject length, colon usage in the subject line, and body line length. This helps maintain consistency and readability in the project's history.

@github-actions github-actions bot added CI size/S Small PR labels Jul 3, 2025
@szymon-czapracki szymon-czapracki changed the title Add GitHub Action to validate commit message style ci: Add GitHub Action to validate commit message style Jul 3, 2025
@szymon-czapracki szymon-czapracki force-pushed the commit_style_wrkfl branch 2 times, most recently from 9cf8c44 to 0c7d05f Compare July 8, 2025 09:28
Copy link
Contributor

@sjanc sjanc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indentation is not valid

Invalid workflow file: .github/workflows/compliance_check.yml#L110
The workflow is not valid. .github/workflows/compliance_check.yml (Line: 110, Col: 1): Unexpected value 'commits-check'

@szymon-czapracki szymon-czapracki requested a review from sjanc July 9, 2025 09:29
@szymon-czapracki szymon-czapracki force-pushed the commit_style_wrkfl branch 2 times, most recently from 3ee120f to ed24707 Compare July 9, 2025 11:57
This workflow checks all commits between the current branch and
upstream/master. It enforces subject length, colon usage in the
subject line, and body line length. This helps maintain consistency
and readability in the project's history.
@sjanc sjanc merged commit 3bb2671 into apache:master Jul 9, 2025
21 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants