-
Notifications
You must be signed in to change notification settings - Fork 844
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
base: master
Are you sure you want to change the base?
Conversation
0770006
to
127107a
Compare
d17b7ab
to
a3f84da
Compare
well late :D will need rebase. |
a3f84da
to
5651995
Compare
rebased, removed string templates and fixed a bug |
java: [ '17' ] | ||
java: [ '23' ] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I hope this doesn't cause issues, but we will see next time this PR fully builds and all paperwork checks run, I removed the no-build label now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this seems to work. But I might change it back anyway. Probably not worth the risk to run sig test-tests on JDK 23.
LGTM |
- simple script to check if all commits have proper email addresses - looks for empty or single word subject messages - warns about missing blank line after subject - logs errors/warnings to the github actions summary page
5651995
to
1e4d17b
Compare
removed the test commits and updated the error messages a bit |
boolean checkBlankLineAfterSubject(String blank, int i) { | ||
if (!blank.isBlank()) { | ||
println("::warning::blank line after subject recommended in commit " + i + " (subject over 50 char limit?)"); | ||
// return false; | ||
} | ||
return true; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i am not sure about this. Will anyone even look at the warnings? Should I turn this off for now?
I suppose a better place for checks like this would be the NB commit window ;)
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.
this adds:
output in workflow summary:
output in job log: