Skip to content
This repository has been archived by the owner on Dec 5, 2022. It is now read-only.

Option to explicitly set the committer #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jerrykan
Copy link

It isn't always clear who is the person or app that initiated the workflow (ie. GITHUB_ACTOR) and it may be desirable to explicitly set who the committer is when the action is run.

This commit provides the option to set the committer using the GH_USER_NAME and/or GH_USER_EMAIL environment variables. If neither environment variable is set it will default to the original behaviour of just using GITHUB_ACTOR.

A use-case for this functionality can be found at upptime/upptime#118

It isn't always clear who is the person or app that initiated the
workflow (ie. `GITHUB_ACTOR`) and it may be desirable to explicitly set
who the committer is when the action is run.

This commit provides the option to set the committer using the
`GH_USER_NAME` and/or `GH_USER_EMAIL` environment variables. If neither
environment variable is set it will default to the original behaviour of
just using `GITHUB_ACTOR`.
@AnandChowdhary
Copy link

@maxheld83, this would be a wonderful addition to your action and will solve many use cases for us! Is it possible to have a quick review here? 😄

@AnandChowdhary
Copy link

Pinging @maxheld83 again 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants