-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat: add configurable user and email #501
Conversation
1157b46
to
f0fbaf7
Compare
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.
@Ninerian Thanks for extracting the PR!
I'd like to keep the backward compatibility, so how about changing the default values like this?
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
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.
Thank you! I request more trivial changes in the descriptions.
Other than that, everything seems to be fine. 😄
And, can you merge the |
Co-authored-by: Masafumi Koba <473530+ybiquitous@users.noreply.github.com>
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.
@Ninerian Thank you! LGTM 👍🏼
v3.3.0 released! 🚀 |
What have I done
Added
github_user
andgithub_email
inputsWhy was this done
I discovered the problem that checks where not running, when the PR is created by the Github Actions user. Using a custom one solved the problem