Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

github: add new checklist workflow #1570

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

VexedUXR
Copy link
Contributor

Add a new 'checklist' workflow that checks the commit messages on pull
requests. Currently, the workflow creates a comment on the pull request
if any of these conditions are hit:

  • Missing Signed-off-by
  • Malformed Signed-off-by
  • Bad email (i.e *noreply*)

A test can be found here: VexedUXR#6
The comment is edited as the pull request is updated.
The comment get updated to:

Thank you for taking the time to contribute to FreeBSD!
All issues resolved.

When all issues are resolved.
If no issues are found in the first place, then no comment is added.

Add a new 'checklist' workflow that checks the commit messages on pull
requests. Currently, the workflow creates a comment on the pull request
if any of these conditions are hit:
- Missing Signed-off-by
- Malformed Signed-off-by
- Bad email (i.e *noreply*)

Signed-off-by: Ahmad Khalifa <[email protected]>
Both the bad email check and the signoff check are handled in the
"checklist" github workflow now.

Signed-off-by: Ahmad Khalifa <[email protected]>
@VexedUXR VexedUXR requested a review from bsdimp as a code owner January 12, 2025 06:24
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.

1 participant