-
Notifications
You must be signed in to change notification settings - Fork 98
docs/contribution: added link to docker installation for MacOS. #478
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
base: main
Are you sure you want to change the base?
docs/contribution: added link to docker installation for MacOS. #478
Conversation
The current documentation doesn't provide details on how to install unikraft within macos. With the new update, it requires installing docker through .dmg file from the official docker website, the updated documentation will allow users to be able to install docker for MacOS without running into errors. Signed-off-by : Ayan Das dasa60196@gmail.com
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DeveloperMindset123 please remove the search-meta.json
and package-lock.json
files from the commit, and see the inline comment.
Also wrap the commit description at 70-80 characters per line.
@@ -60,6 +60,8 @@ You can do that by following the instructions [here](https://github.com/docker/d | |||
```console | |||
curl -fsSL https://get.docker.com/ | sh | |||
``` | |||
### Important Note |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
### Important Note | |
### Important Note | |
Add blank linkes here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi, thanks for the contribution, please see inline notes and also remove the diff from the search-meta.json
and package-lock.json
.
@@ -60,6 +60,8 @@ You can do that by following the instructions [here](https://github.com/docker/d | |||
```console | |||
curl -fsSL https://get.docker.com/ | sh | |||
``` | |||
### Important Note | |||
If you happen to be using `MacOS`, please refer to the following documentation [here](https://docs.docker.com/desktop/setup/install/mac-install/) instead. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove quotes around macOS:
- `MacOS`
+ macOS
The current documentation doesn't provide details on how to install unikraft within macos. With the new update, it requires installing docker through .dmg file from the official docker website, the updated documentation will allow users to be able to install docker for MacOS without running into errors.
Signed-off-by : Ayan Das dasa60196@gmail.com