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

Revert "fix: appease the git directory ownership check" #10

Merged
merged 1 commit into from
Jun 15, 2023

Conversation

hwkns
Copy link

@hwkns hwkns commented Jun 15, 2023

Reverts #9

error: could not lock config file /etc/gitconfig: Permission denied

Continuing to investigate... Seems like we shouldn't have to run this command ourselves anyway, because...

  1. the checkout@v3 action has a default-true option to add the safe directory git config...
    # Add repository path as safe.directory for Git global config by running `git
    # config --global --add safe.directory <path>`
    # Default: true
    set-safe-directory: ''
  1. the gitleaks docker image has a RUN step to set it as well...
    RUN /bin/sh -c git config --global --add safe.directory '*' # buildkit

Working my way through the many comments on actions/runner-images#6775

@hwkns hwkns merged commit 94533d9 into master Jun 15, 2023
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants