ci: Add Conventional Commit Lint #821
Conversation
arielvalentin
force-pushed
the
add-commitlint-gh-action
branch
from
989a3b3 to
0a4b299
Compare
arielvalentin
force-pushed
the
add-commitlint-gh-action
branch
from
4278776 to
b54c6df
Compare
arielvalentin
changed the title
arielvalentin
force-pushed
the
add-commitlint-gh-action
branch
from
86f22dc to
42581d7
Compare
arielvalentin
force-pushed
the
add-commitlint-gh-action
branch
from
6c639b6 to
5359c0b
Compare
arielvalentin
changed the title
arielvalentin
changed the title
|
@scbjans @kaylareopelle I got something working but it required two different GitHub actions to get what I wanted.
Now trying merge commit messages... |
|
yup... seems like merge commits are ok https://github.com/open-telemetry/opentelemetry-ruby-contrib/actions/runs/7551640637/job/20559186184?pr=821 |
arielvalentin
requested review from
fbogsany,
mwear,
robertlaurin,
dazuma,
ericmustin,
ahayworth,
plantfansam,
robbkidd,
simi,
kaylareopelle and
xuan-cao-swi
as code owners
There was a problem hiding this comment.
@arielvalentin - Yay! I'm so glad it's working. I have two questions.
| fix | ||
| feat | ||
| chore | ||
| ci | ||
| docs | ||
| style | ||
| refactor | ||
| perf | ||
| test |
There was a problem hiding this comment.
| fix | |
| feat | |
| chore | |
| ci | |
| docs | |
| style | |
| refactor | |
| perf | |
| test | |
| chore | |
| ci | |
| docs | |
| feat | |
| fix | |
| perf | |
| refactor | |
| style | |
| test |
Mind if we alphabetize this list?
Also, the list in commitlint.config.js also includes revert and squash. Should those be here too?
Yoohoo! Is there a limit on the number of GH actions we can use? |
|
closed in favor of #800 |
This is an attempt to add conventional commit lint.
It is an alternative to #800