config: Added checkstyle's commit message CI test

[MiriyalaJayanth-19]

Addded a ci job that matches the commit message format to that of CheckStyle's.

Commit message must match any one of the following patterns:
- ^Issue #\d+: .*$
- ^Pull #\d+: .*$
- ^(minor|config|infra|doc|spelling|dependency): .*$
- Length : max 200 characters

[romani]

Can we reuse https://github.com/marketplace/actions/gitlint-action
Or similar already existing action ?

[MiriyalaJayanth-19]

I think its better we use gitlint-action and modify it to checkstyle's rules, made the changes.

[romani]

I still see bunch of bash scripting instead of action reusage

[MiriyalaJayanth-19]

@romani made the changes , added .gitlint file in project root repo with checkstyle commit rules and linter.yml in /.github/workflow

[romani]

CI is red

[romani]

add new lines to all files you added

[MiriyalaJayanth-19]

The CI now uses gitlint with a configuration aligned to Checkstyle’s commit message rules. It validates:

Required patterns (Issue #, Pull #, or allowed prefixes)
Max length (≤ 200 characters)
No trailing punctuation or whitespace
I also scoped the validation to only the commits from the current PR by keeping base.sha..HEAD , so it doesn’t affect already merged commits.

I added new line at end of both the files , i think github is not showing it.

Thanks for guiding me patiently . It means a lot .

[romani]

Code is good, please add new commit with any update in code and bad commit message, I want to see CI failure.

[MiriyalaJayanth-19]

@romani should i add a section in linter.yml file that outputs clear error message upon ci failure like in checkstyle.

[romani]

Commit b616b06b98:
1: T7 Title does not match regex (^(Issue #\d+: .*|Pull #\d+: .*|(minor|config|infra|doc|spelling|dependency): .*)$): "added checkstyles commit-message ci test"
Error: Process completed with exit code 1.

https://github.com/checkstyle/patch-filters/actions/runs/24703915425/job/72252838475?pr=388

all good.

please remove text commit, lets merge this PR.

[MiriyalaJayanth-19]

@romani ping

[romani]

Thanks a lot.

Let's see how this works in long run.