Skip to content

appending cmd tab in trying out bind mounts section to work Closes #18242#18243

Merged
craig-osterhout merged 13 commits intodocker:mainfrom
arminshfatemi:main
Oct 5, 2023
Merged

appending cmd tab in trying out bind mounts section to work Closes #18242#18243
craig-osterhout merged 13 commits intodocker:mainfrom
arminshfatemi:main

Conversation

@arminshfatemi
Copy link
Contributor

Proposed changes

Related issues (optional)

@netlify
Copy link

netlify bot commented Sep 20, 2023

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit dcbe1d0
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/651ed7c616c46d00083a17eb
😎 Deploy Preview https://deploy-preview-18243--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 configuration.

@craig-osterhout craig-osterhout added the area/get-started Relates to get started and onboarding docs label Sep 20, 2023
Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

Thanks for adding the command.
Commented on a few style nits and a numbering issue.

arminshfatemi and others added 11 commits September 30, 2023 14:08
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Co-authored-by: Craig Osterhout <103533812+craig-osterhout@users.noreply.github.com>
Copy link
Contributor Author

@arminshfatemi arminshfatemi left a comment

Choose a reason for hiding this comment

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

looks like this line was ok

Copy link
Contributor Author

@arminshfatemi arminshfatemi left a comment

Choose a reason for hiding this comment

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

looks like dont need to change

Copy link
Contributor Author

@arminshfatemi arminshfatemi left a comment

Choose a reason for hiding this comment

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

looks like it didnt need to change

Copy link
Contributor

@craig-osterhout craig-osterhout left a comment

Choose a reason for hiding this comment

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

Thank you. LGTM

@craig-osterhout craig-osterhout merged commit 217922c into docker:main Oct 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/get-started Relates to get started and onboarding docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants