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

fix: set workspace as safe directory #183

Merged
merged 1 commit into from
Jun 27, 2022
Merged

fix: set workspace as safe directory #183

merged 1 commit into from
Jun 27, 2022

Conversation

pascaliske
Copy link
Contributor

@pascaliske pascaliske commented Jun 26, 2022

Hi @mhausenblas,

It seems that with newer Git versions (which is the case due to the base image update) the following error occurs in a workflow with steps inside containers:

fatal: unsafe repository ('/github/workspace' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /github/workspace

To workaround this issue I've added a git config line inside the Docker entry point script.

For more information see here: actions/checkout#766

@pascaliske pascaliske marked this pull request as ready for review June 26, 2022 19:19
More information here: actions/checkout#766

Signed-off-by: Pascal Iske <info@pascaliske.dev>
pascaliske added a commit to pascaliske/infrastructure that referenced this pull request Jun 27, 2022
@mhausenblas
Copy link
Owner

That should fix #184

@mhausenblas
Copy link
Owner

Thanks a bunch for this fix @pascaliske

@mhausenblas mhausenblas merged commit b1bb7cb into mhausenblas:master Jun 27, 2022
@pascaliske pascaliske deleted the feature/git-unsafe-workaround branch June 27, 2022 16:16
andi34 pushed a commit to andi34/mkdocs-deploy-gh-pages that referenced this pull request Jan 17, 2024
More information here: actions/checkout#766

Signed-off-by: Pascal Iske <info@pascaliske.dev>
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.

2 participants