Skip to content

ci: add commitlint workflow for PR commit messages - #4902

Open
ginglis13 wants to merge 1 commit into
bottlerocket-os:developfrom
ginglis13:add-commitlint
Open

ci: add commitlint workflow for PR commit messages#4902
ginglis13 wants to merge 1 commit into
bottlerocket-os:developfrom
ginglis13:add-commitlint

Conversation

@ginglis13

Copy link
Copy Markdown
Contributor

Issue number:

n/a

Description of changes:

Add linting to commit messages for this repo. We've had some churn (and misses) the last few weeks on commit style, so let's enforce it via CI. The linting checks for scope: message style commits. This is slightly different than conventional commits, which are more the form fix(scope): message or chore(scope): message, but the linting is based on our own conventions and commit history.

Testing done:

Ran locally against last 50 commits. Only latest commit 0cef809 failed because it is missing a colon.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Signed-off-by: Gavin Inglis <giinglis@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants