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

Added note for using bind mount on Windows host systems #14312

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

PyroGenesis
Copy link

Proposed changes

Added a note to the bind mounts page pointing developers to first turn on File Sharing in Docker Desktop and also to a page highlighting path differences for Volumes in Windows.

This information was not apparent to me and so I want to make it clear for others.

@netlify
Copy link

netlify bot commented Feb 25, 2022

✔️ Deploy Preview for docsdocker ready!

🔨 Explore the source changes: 07013a2

🔍 Inspect the deploy log: https://app.netlify.com/sites/docsdocker/deploys/621a762704cc1100071d6ba6

😎 Browse the preview: https://deploy-preview-14312--docsdocker.netlify.app

@craig-osterhout craig-osterhout added the area/storage Relates to storage, volumes label Aug 4, 2022
@docker-robott
Copy link
Collaborator

Thanks for the pull request. We'd like to make our product docs better, but haven’t been able to review all the suggestions.
As our docs have also diverged, we do not have the bandwidth to review and rebase old pull requests.

If the updates are still relevant, review our contribution guidelines and rebase your pull request against the latest version of the docs, then mark it as fresh with a /remove-lifecycle stale comment.
If not, this pull request will be closed in 30 days. This helps our maintainers focus on the active pull requests.

Prevent pull requests from auto-closing with a /lifecycle frozen comment.

/lifecycle stale

@netlify
Copy link

netlify bot commented Jan 25, 2023

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit acd81e3
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/64594bff33d2f40008ca7957
😎 Deploy Preview https://deploy-preview-14312--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@PyroGenesis PyroGenesis force-pushed the patch-1 branch 3 times, most recently from 4d75648 to 91f2986 Compare January 30, 2023 23:57
@PyroGenesis
Copy link
Author

/remove-lifecycle stale

@docker-robott
Copy link
Collaborator

Thanks for the pull request. We'd like to make our product docs better, but haven’t been able to review all the suggestions.
As our docs have also diverged, we do not have the bandwidth to review and rebase old pull requests.

If the updates are still relevant, review our contribution guidelines and rebase your pull request against the latest version of the docs, then mark it as fresh with a /remove-lifecycle stale comment.
If not, this pull request will be closed in 30 days. This helps our maintainers focus on the active pull requests.

Prevent pull requests from auto-closing with a /lifecycle frozen comment.

/lifecycle stale

@PyroGenesis
Copy link
Author

/remove-lifecycle stale

/lifecycle frozen

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/storage Relates to storage, volumes lifecycle/frozen
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants