CI: Add PR title check workflow#16101
Conversation
manuzhang
force-pushed
the
add-pr-title-check-workflow
branch
from
b0d24af to
f505653
Compare
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
manuzhang
force-pushed
the
add-pr-title-check-workflow
branch
from
f505653 to
dcb7e50
Compare
|
This is to address the confusion on PR title format and I think it would be good to "enforce" it through a workflow rather than human check. WDYT? @rdblue @jbonofre @kevinjqliu |
kevinjqliu
left a comment
kevinjqliu
left a comment
There was a problem hiding this comment.
I see https://github.com/amannn/action-semantic-pull-request as a popular option (and it's allowlisted by ASF already).
I've never used it, so I'm not sure if it's better.
One concern with using a GitHub workflow to check the PR title: if I open a PR with an incorrectly formatted title, the check will fail, but there's no way to re-trigger CI without pushing an empty commit.
Co-authored-by: Kevin Liu <kevinjqliu@users.noreply.github.com>
Co-authored-by: Copilot <copilot@github.com>
kevinjqliu
changed the title
kevinjqliu
changed the title
|
verified by changing this pr's title https://github.com/apache/iceberg/actions/runs/25606924463/job/75170331982?pr=16101 |
kevinjqliu
left a comment
kevinjqliu
left a comment
There was a problem hiding this comment.
LGTM, thanks for adding this.
Let's give it a try, we can revert if there's too much friction
3 participants
No description provided.