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

Try to validate commit headers. #6630

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Conversation

mbien
Copy link
Member

@mbien mbien commented Oct 28, 2023

Back when I setup the paperwork job, I didn't know that gh can generate patch files yet. The problem was to figure out what commits are actually new in the PR. I gave it a few attempts but got distracted with more important things.

in summary:
image
in job:
image

@mbien mbien added CI Continuous Integration ci:no-build [ci] disable CI pipeline labels Oct 28, 2023
@mbien mbien requested a review from ebarboni October 28, 2023 00:29
@mbien mbien force-pushed the ci-email-checker branch 2 times, most recently from 0770006 to 127107a Compare October 28, 2023 20:35
@mbien mbien force-pushed the ci-email-checker branch 3 times, most recently from a562b94 to d17b7ab Compare December 15, 2023 14:33
mbien and others added 5 commits December 15, 2023 17:16
 - simple script to check if all commits have proper email addresses
 - looks for empty or single word subject messages
 - warns about missing blank lines after subject
 - prints errors in the github actions error log format
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci:no-build [ci] disable CI pipeline CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant