chore: enforce PR title format with PRLint Reloaded#8394
chore: enforce PR title format with PRLint Reloaded#8394paulbalandan merged 1 commit intocodeigniter4:developfrom
Conversation
|
I'm not sure what others will say, but I believe we were thinking of going in the opposite direction in terms of tightening PR requirements. Potentially speaking... every commit would be checked, or only the most recent? |
|
This approach only checks PR titles upon creation, not commit messages involved in the PR, correct? If so, I think this is quite useful and not a hurdle keeping beginners from contributing. |
|
Ok, maybe I didn't fully understand it. If only the PR title is checked, I would be fine with it. |
paulbalandan
force-pushed
the
prlint-reloaded
branch
from
a4509ff to
817c482
Compare
lonnieezell
left a comment
lonnieezell
left a comment
There was a problem hiding this comment.
If I'm understanding how and when this works then I think this is fine. Will also make compiling our changelogs simpler.
|
@lonnieezell thanks for the approval! can you install the Github app to the org so this can be merged? |
Sure. How would I do this? |
|
@paulbalandan Should be installed now for the codeigniter4 org. |
paulbalandan
force-pushed
the
prlint-reloaded
branch
from
817c482 to
53d1382
Compare
|
Rebased to test the app. And all passed 🎉 |
5 participants
Description
Enforce Conventional Commit format of PR title using PRLint Reloaded
Step 1: Allow access to the
codeigniter4org then to this repo for the app (I have no rights to do so)Step 2: Merge this PR to let the configuration kick in
For the regex reference: https://regex101.com/r/Yh1iT9/1
Checklist: