Description
It would be great if this action would also set the email
and name
. In order to do other git operations it is required. I am adding this to my actions to get around it:
steps:
- uses: actions/checkout@v1
- name: Setup git user
run: |
git config --global user.name "$(git --no-pager log --format=format:'%an' -n 1)"
git config --global user.email "$(git --no-pager log --format=format:'%ae' -n 1)"
Unfortunately I don't have access to the user email. It seems like these can come from the account in a more clean way here, but I guess I could also create my own action. Since this is so new I am not sure what the best way forward is so thought I would post here.
Activity
TingluoHuang commentedon Aug 16, 2019
@chrispat for feedback.
eine commentedon Oct 25, 2019
This would be useful for users willing to update
gh-pages
hosted on separate repos using a deployment key (e.g. https://github.com/1138-4EB/vunit/blob/migrate-to-gha/.github/workflows/push.yml#L63-L74). This is specially so, becauseGITHUB_TOKEN
's scope is limited and personal access tokens are not scoped to a single target repo.wesleytodd commentedon Oct 27, 2019
This was actually my first use case for this!
https://github.com/pkgjs/gh-pages/blob/master/entrypoint.sh#L4-L5
fregante commentedon Oct 31, 2019
I recently wrote an action that sets up the user and email the @action user, so you can also push back to the repo, as long as you use the v2 of
actions/checkout
https://github.com/fregante/setup-git-user
Old action
I recently wrote an action that sets up the user, email and token in one go, so you can also push back to the repo.https://github.com/fregante/setup-git-token
eine commentedon Dec 16, 2019
@ericsciple, should this issue be closed now that #107 is merged? It seems to be included in v2: https://github.com/actions/checkout/commits/v2
wesleytodd commentedon Dec 18, 2019
So that PR is a bit hard to grok on my phone (even with the improvements in the GitHub app), but it is unclear if that sets the
name
andemail
. This issue is specifically about that need. Does that PR solve this use case indirectly?eine commentedon Dec 18, 2019
@wesleytodd, see:
checkout/action.yml
Lines 12 to 17 in 3537747
It seems to me that this issue was/is about "do other (authenticated) git operations"; precisely update gh-pages (which was/is your "first use case for this"). Explicitly setting the user and email would have been an approach. Using the token to do so is just another suitable solution.
Moreover, I think it would be desirable to support optionally using deployment keys instead of tokens (be it the default or a PAT). This is because default tokens do not allow to update other repos; while PATs provide too many permissions. Nonetheless, even if keys were supported, I believe that setting the user and email would still be a side effect.
Therefore, I think that this issue should be either closed or repurposed to account for the new feature in
master
.wesleytodd commentedon Dec 19, 2019
Awesome, thanks for the more clear explanation! I have never used the method I see in that PR, so one last question I have:
Does using a token for git auth also stop git from complaining about no user or email when committing?
eine commentedon Dec 19, 2019
Although I'm not sure about that, I believe it should. The usefulness of the feature would be very limited otherwise.
Anyway, even if it works, there is currently a bug that prevents gh-pages from being updated when the token is used: actions/toolkit#247. Hence, you might want to keep using a deployment key.
HDouss commentedon Jan 28, 2020
No, that does not stop git from complaining. You still have to
git config
name and email. All what it does is that commits and pushes are well authenticated, but you should provide name and email before that. And yes, that limits the usefulness as @eine said.ericsciple commentedon Feb 13, 2020
+1 i didnt realize the built-in GITHUB_TOKEN has push permission... i'll open an adr shortly...
67 remaining items