Skip to content

docs: fix minor grammar errors#25372

Merged
dvdksn merged 1 commit into
docker:mainfrom
rafaelcoelhox:docs-grammar
Jun 17, 2026
Merged

docs: fix minor grammar errors#25372
dvdksn merged 1 commit into
docker:mainfrom
rafaelcoelhox:docs-grammar

Conversation

@rafaelcoelhox

Copy link
Copy Markdown
Contributor

Description

Fix several small grammar errors across the docs:

  • GitHub Actions CI guides (Python and Ruby): This workflow is runsThis workflow runs; tab of you GitHub repositoryof your GitHub repository; [Login to Docker Hub] link text → [Log in to Docker Hub], matching the adjacent "Set up Docker Buildx" step
  • build/ci/github-actions/configure-builder: a node in an underlying step of you workflowof your workflow
  • Bind mounts page (get started): remove a stray article in image alt text — inside the a containerinside the container
  • Database pre-seeding guide: let you access to the Postgres databaseletting you access the Postgres database; It also defineIt also defines

Reviews

  • Technical review
  • Editorial review
  • Product review

@rafaelcoelhox rafaelcoelhox requested a review from dvdksn as a code owner June 16, 2026 18:51
@netlify

netlify Bot commented Jun 16, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit dcfd3df
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a3291fa695f4200083637eb
😎 Deploy Preview https://deploy-preview-25372--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-actions github-actions Bot added area/build Relates to Dockerfiles or docker build command area/get-started Relates to get started and onboarding docs area/guides labels Jun 16, 2026
@rafaelcoelhox

rafaelcoelhox commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Heads-up for reviewers (cc @craig-osterhout):

Two files in this PR are being removed in #25373:

  • guides/python/configure-github-actions.md
  • guides/ruby/configure-github-actions.md

The other three files don't overlap with that PR:

  • build/ci/github-actions/configure-builder.md
  • get-started/docker-concepts/running-containers/sharing-local-files.md
  • guides/pre-seeding.md

Happy to drop the two GitHub Actions guide files from this PR to avoid a modify/delete conflict — just let me know if you'd prefer that, or if you'd rather merge this one first.

@dvdksn

dvdksn commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

hey @rafaelcoelhox thanks for raising this PR - yeah if you could drop the updates to the pages that are being removed, that would be grand 🙏

  - Python/Ruby GitHub Actions CI guides: 'is runs' -> 'runs', 'of you' ->
    'of your', and 'Login to Docker Hub' link text -> 'Log in to Docker Hub'
  - configure-builder: 'of you workflow' -> 'of your workflow'
  - bind-mount page: remove stray article ('the a container' -> 'the container')
  - pre-seeding guide: 'let you access to' -> 'letting you access', 'define' -> 'defines'
@rafaelcoelhox

Copy link
Copy Markdown
Contributor Author

Done! dropped the changes to the two GitHub Actions guide pages (`python/configure-github-actions.md` and `ruby/configure-github-actions.md`). Now only touches the three non-overlapping files. Thanks @dvdksn!

@dvdksn dvdksn left a comment

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.

Thanks 🚀

@dvdksn dvdksn merged commit d22eea2 into docker:main Jun 17, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build Relates to Dockerfiles or docker build command area/get-started Relates to get started and onboarding docs area/guides

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants