style: check pr title with commitlint

[andysim3d]

[Closes/Fixes] #

Proposed Changes

test pr title check.

[dancoombs]

Should we just do this ourselves? So that we can make sure its the check that runs against the commits? Similar to how its done in AA-SDK.

[andysim3d]

you mean check-in a binary/tool to run the semantic lint check?

I find this one on action marketplace and thinking 3 benefits of using it:

• the github action get updates if in the future semantic pull request extended.
• the binary checks commits semantic is not related to our main logic. we use cocogitto/cocogitto-action@v3 to check it.
• this repo is open sourced, granting read access of github action is low risk.

I have no objections to use our own tool to check semantic commit/pr title and merge 2 checks in one to keep the check consistent.

[dancoombs]

Got it. My concern here would be if action-semantic-pull-request and cocogitto-action diverge in their enforcement of conventional commits then we could get commits in that are invalid to cocogitto-action and thus cause downstream PRs to fail CI.

If we don't think thats a problem at the moment, we can just address it if it becomes one.

[dancoombs]

Should we sync our config with the one from the action github?

https://github.com/amannn/action-semantic-pull-request?tab=readme-ov-file#installation

[andysim3d]

Should we sync our config with the one from the action github?

https://github.com/amannn/action-semantic-pull-request?tab=readme-ov-file#installation

the example is based on multi-forks repo. we use multi-branch so the trigger and read permission are slightly different.