-
Notifications
You must be signed in to change notification settings - Fork 578
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
GHA: complain if PR adds commits from people not yet listed in ./AUTHORS #9911
Conversation
74bbb03
to
a8d4e9d
Compare
Not critically necessary, but should prevent stuff like #9697 (comment). |
a8d4e9d
to
b71cff5
Compare
not to have to update ./AUTHORS or .mailmap after merging.
b71cff5
to
20fcf26
Compare
Thank you for your pull request and welcome to our community. We could not parse the GitHub identity of the following contributors: Lol cat.
|
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.
Looks fine in the failing case, please remove the second commit so that we can also see it working if everything is fine.
20fcf26
to
8025e02
Compare
not to have to update ./AUTHORS or .mailmap after merging.
This substitutes ProBot functionality.