Skip to content

readme: add notes about Docker for MacOS users #29

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

Merged
merged 2 commits into from
Jul 22, 2018

Conversation

tennix
Copy link
Member

@tennix tennix commented Jul 20, 2018

This PR updates README to inform MacOS users using Docker for Mac. It closes #23 and closes #28
PTAL @lilin90

@tennix tennix requested a review from lilin90 July 20, 2018 03:53
README.md Outdated
@@ -5,6 +5,8 @@
* Docker >= 16.10
* Docker Compose >= 1.6.0

*Note*: [Legacy Docker Toolbox](https://docs.docker.com/toolbox/toolbox_install_mac/) users should migrate to [Docker for Mac](https://store.docker.com/editions/community/docker-ce-desktop-mac), since it is tested that tidb-docker-compose can't started on Docker Toolbox and Docker Machine.
Copy link
Member

@lilin90 lilin90 Jul 20, 2018

Choose a reason for hiding this comment

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

  • Please follow our note format: > **Note:**
  • should -> must
  • can't started -> cannot be started

Copy link
Member

@lilin90 lilin90 left a comment

Choose a reason for hiding this comment

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

LGTM

@lilin90 lilin90 changed the title add notes about Docker for MacOS users readme: add notes about Docker for MacOS users Jul 22, 2018
@lilin90 lilin90 merged commit 5a2a66d into master Jul 22, 2018
@lilin90 lilin90 deleted the tennix/docker-for-mac branch July 22, 2018 09:24
tennix added a commit that referenced this pull request Aug 15, 2018
readme: add notes about Docker for MacOS users
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