Skip to content

Conversation

@dhruvdutt
Copy link
Contributor

@dhruvdutt dhruvdutt commented May 26, 2021

Running this action inside a self-hosted GitHub runner that has an Alpine equivalent barebones environment, the changeset action fails since it's trying to set git user & email on global config that looks for $HOME/.gitconfig.

Self-hosted instances might also have a shared VM running multiple jobs, updating global git config might pollute the global configs which is not expected.

image

@dhruvdutt dhruvdutt changed the title Update gitUtils.ts fix: remove git global config flag May 26, 2021
@dhruvdutt
Copy link
Contributor Author

@mitchellhamilton @Andarist Can you please take a look at this?

Copy link
Member

@Andarist Andarist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, @mitchellhamilton do you see any risks associated with this? I couldn't find any

@Andarist Andarist changed the base branch from master to main December 22, 2021 08:53
@Andarist Andarist mentioned this pull request Dec 23, 2021
@Andarist
Copy link
Member

This got merged in as part of this PR #128 because I couldn't push to this fork.

@Andarist Andarist closed this Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants