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) Add Dockerfile git safe directory for/app #1120

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

pauby
Copy link
Member

@pauby pauby commented Jan 16, 2025

Description Of Changes

The docker container doesn't without marking the /app directory as safe.

Motivation and Context

Added the git safe directory command to mark /app as safe. Similar change went into blog.

Testing

  • I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.
    • Issue #

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

N/A

@pauby pauby requested a review from st3phhays January 16, 2025 13:07
@st3phhays st3phhays force-pushed the fix/add-git-safe-directory branch from a87fbed to 18887c7 Compare January 21, 2025 17:13
Copy link
Member

@st3phhays st3phhays left a comment

Choose a reason for hiding this comment

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

This LGTM, thanks for getting this fix in!

@st3phhays st3phhays merged commit 69c7f32 into chocolatey:master Jan 21, 2025
1 check 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
Development

Successfully merging this pull request may close these issues.

2 participants