Skip to content

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

DeveloperMindset123
Copy link

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

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
@razvand razvand requested a review from StefanJum February 23, 2025 07:42
@razvand razvand self-assigned this Feb 23, 2025
@razvand razvand added the documentation Improvements or additions to documentation label Feb 23, 2025
Copy link
Member

@StefanJum StefanJum left a 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
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
### Important Note
### Important Note

Add blank linkes here

Copy link
Member

@nderjung nderjung left a 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.
Copy link
Member

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

4 participants