Skip to content

Conversation

@ArthurFlag
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Jun 4, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 134c4ce
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/68400433aec85000089a598a
😎 Deploy Preview https://deploy-preview-22760--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 project configuration.

enabled on the default port (12434).

{{< /tab >}}
{{< tab name="Docker CE">}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
{{< tab name="Docker CE">}}
{{< tab name="Docker CE">}}

Docker CE is the internal name for Docker Desktop 🤔 do we mean Docker Engine?

Copy link
Member

@kiview kiview Jun 4, 2025

Choose a reason for hiding this comment

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

Docker CE is the internal name for Docker Desktop

Not sure where you are getting this from, but that's not correct.
Docker CE is Docker Community Edition, our Docker distribution for Linux, consisting of Docker Engine (Moby), Docker CLI and BuildKit.

From our docs (which are fuzzy, agreed):

This section describes how to install Docker Engine on Linux, also known as Docker CE. Docker Engine

Since the docs are about adding Docker Model Runner to Docker CE (as a plugin), referring to Docker CE should be the right language.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ok yeh - not Docker Desktop, coffee hadn't hit the ol' brain matter yet.

We've never used Docker CE in docs or marketing to refer to Docker Engine on Linux (apart from this one sentence) so whilst technically correct, this is not how users know it and I fear this would add unnecessary confusion

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's indeed Docker Engine, as I also used in the top section... I also went through the confusion between CE/Engine last week 🫠

Copy link
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

LGTM

@ArthurFlag ArthurFlag merged commit cd5757d into docker:main Jun 4, 2025
14 checks passed
@ArthurFlag ArthurFlag deleted the ENGDOCS-2692-update-api-info-for-ce branch June 4, 2025 08:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants