Skip to content

Fix grammar in environment variables section - #23620

Closed
ahopp wants to merge 2 commits into
docker:mainfrom
ahopp:main
Closed

Fix grammar in environment variables section#23620
ahopp wants to merge 2 commits into
docker:mainfrom
ahopp:main

Conversation

@ahopp

@ahoppahopp commented Oct 28, 2025

Copy link
Copy Markdown
Contributor

Summary

Corrects grammar error in the environment variables section of the Docker CLI documentation. Specifically, the subject of the sentence is "list" (singular), not "environment variables" (plural). The phrase "of environment variables" is a prepositional phrase that modifies "list" but doesn't change the fact that the subject is singular. So updating.

Changes

Changed "The following list of environment variables are supported" to "The following list of environment variables is supported"

Related Issue

Fixes#23616

Change "are supported" to "is supported" since the subject is "list" (singular).
Fixesdocker#23616
@netlify

netlifyBot commented Oct 28, 2025

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

NameLink
🔨 Latest commitb8d0295
🔍 Latest deploy loghttps://app.netlify.com/projects/docsdocker/deploys/69013c363716220008913b90
😎 Deploy Previewhttps://deploy-preview-23620--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.

@github-actionsgithub-actionsBot added the area/cli Relates to the CLI client label Oct 28, 2025
Add clarification that the application may take 10-15 seconds to fully start
after running docker compose up. This helps users understand why the app may
not be immediately accessible at localhost:3000.
Fixesdocker#23500
@github-actionsgithub-actionsBot added the area/get-started Relates to get started and onboarding docs label Oct 28, 2025
@ahopp

Copy link
Copy Markdown
ContributorAuthor

@sarahsanders-docker@dvdksn Can you take a look?

### Environment variables

The following list of environment variables are supported by the `docker` command
The following list of environment variables is supported by the `docker` command

@dvdksndvdksnOct 31, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

good catch but unfortunately this file is generated, from this: https://github.com/docker/cli/blob/master/docs/reference/commandline/docker.md

And reading this just now, maybe we could even clarify what we mean here with:

The following environment variables configure the Docker CLI:

Would you mind submitting a PR to docker/cli and dropping it from here?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Closing now. Thanks.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cliRelates to the CLI clientarea/get-startedRelates to get started and onboarding docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wrong grammar

2 participants

@ahopp@dvdksn