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

Modifying Commit Author #119

Closed
creativecreatorormaybenot opened this issue Jan 10, 2021 · 9 comments
Closed

Modifying Commit Author #119

creativecreatorormaybenot opened this issue Jan 10, 2021 · 9 comments
Assignees
Labels
enhancement New feature or request

Comments

@creativecreatorormaybenot

I think this action is really useful 🙌 And you certainly saved the time of many others as well :)

Description

It would be great if one could specify the commit author configuration for the PR.

Existing example

The "Deploy to GitHub Pages" action allows you to do that for example: https://github.com/marketplace/actions/deploy-to-github-pages

See https://github.com/marketplace/actions/deploy-to-github-pages#optional-choices.

@github-actions
Copy link
Contributor

Hi there and welcome to this repository!

A maintainer will be with you shortly, but first and foremost I would like to thank you for taking the time to report this issue. Quality is of the highest priority for us, and we would never release anything with known defects. We aim to do our best but unfortunately you are here because you encountered something we didn't expect. Lets see if we can figure out what went wrong and provide a remedy for it.

@FantasticFiasco
Copy link
Owner

Hi @creativecreatorormaybenot!

Glad you think you saved some time using this action 😄

It is the parameters GIT_CONFIG_NAME and GIT_CONFIG_EMAIL you are thinking of, which currently in this action is hardcoded to github-actions and github-actions@github.com?

@creativecreatorormaybenot
Copy link
Author

@FantasticFiasco Yes, exactly. It would be amazing to sign the commits of course, however, I assume that this would be too complicated.

If you look at the commit in my repo (creativecreatorormaybenot/funvas@039e652), it looks pretty ugly because of two things: 1. it has @github-actions as the PR author (which is okay), however, it has a user without an icon as the commit author. That should likely have my config instead (i.e. my name and email) 👍

@FantasticFiasco
Copy link
Owner

I think we can support git name and e-mail without any breaking changes, and I don't foresee any problems with the implementation either.

Regarding GPG signing, please open a new issue regarding that and we'll see what we can do about it. I think I need your input there, because it would most probably require you to save your private key in GitHub, and I would argue that not everybody would be comfortable doing so.

@creativecreatorormaybenot
Copy link
Author

@FantasticFiasco I agree - I probably do not even want to open an issue because of that.

@FantasticFiasco
Copy link
Owner

@creativecreatorormaybenot, would you be able to update your configuration, and replace <your name> and <your e-mail> with relevant values.

- uses: FantasticFiasco/action-update-license-year@66564cc720675678f03c008d254ebbf5f22609ac
      with:
        token: ${{ secrets.GITHUB_TOKEN }}
        path: '**/LICENSE'
        commitTitle: Update copyright year in LICENSE file(s)
        prTitle: Update copyright year in LICENSE file(s)
        prBody: New Year is here and we want to keep up with time :)
        assignees: creativecreatorormaybenot
        commitAuthorName: <your name>
        commitAuthorEmail: <your e-mail>

If you find that the behavior meets your expectations, I'll continue with releasing this officially.

@creativecreatorormaybenot
Copy link
Author

@FantasticFiasco will do tomorrow - thanks!

@creativecreatorormaybenot
Copy link
Author

creativecreatorormaybenot commented Jan 12, 2021

@FantasticFiasco Worked great! Thank you so much ❤

See creativecreatorormaybenot/wakelock#83 :)

image

@FantasticFiasco
Copy link
Owner

You should be able to use @v2 of the action now, since the feature officially has been released.

Thanks for your feature suggestion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants