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

fix(app/docker): rename distro to Docker Desktop #22

Closed
wants to merge 1 commit into from

Conversation

shirok1
Copy link
Contributor

@shirok1 shirok1 commented Jan 20, 2025

No description provided.

[docker win]
distro = Docker
[docker desktop win]
distro = Docker Desktop
Copy link
Member

Choose a reason for hiding this comment

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

你确定?

image

[docker mac]
distro = Docker
[docker desktop mac]
distro = Docker Desktop
Copy link
Member

Choose a reason for hiding this comment

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

image

看起来不像 Docker Desktop。

@taoky taoky closed this Jan 20, 2025
@taoky
Copy link
Member

taoky commented Jan 20, 2025

可能应该改叫 Docker client(macOS)和 Docker Windows container server and client(Windows)?但是绝对不是 Docker Desktop,这是个有商业授权限制的私有软件,不应该出现在 docker-ce 里面。

@shirok1
Copy link
Contributor Author

shirok1 commented Jan 20, 2025

抱歉脑抽了,看到单个文件 + 仅 Windows + macOS 就以为是 Desktop

是否可以讨论下更好的文案,避免被认为是 Desktop?官方页面对此没有单独的称呼 https://docs.docker.com/engine/install/,也许可以将 distro 改为 Docker Engine

@taoky
Copy link
Member

taoky commented Jan 20, 2025

对应的应该是这个页面:https://docs.docker.com/engine/install/binaries/

但是这么看感觉对应的用途实在太偏门了,Windows 和 macOS 上想跑 Docker 的应该要的绝大部分都不是这个。我在想是否应该整个删掉。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants