Skip to content

docker-images: refresh pages #17901

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 30, 2025
Merged

docker-images: refresh pages #17901

merged 2 commits into from
Aug 30, 2025
+13 −13

Conversation

nelsonfigueroa
Copy link
Member

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The page(s) follow the style guide.
  • The PR title conforms to the recommended templates.

@github-actions github-actions bot added the translation edit Changes to an existing translation page(s). label Aug 30, 2025
spageektti
spageektti approved these changes Aug 30, 2025
View reviewed changes
Copy link
Member

@spageektti spageektti left a comment

Choose a reason for hiding this comment

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

Thanks

SpikeTheDragon40k
SpikeTheDragon40k approved these changes Aug 30, 2025
View reviewed changes
Copy link
Collaborator

@SpikeTheDragon40k SpikeTheDragon40k left a comment

Choose a reason for hiding this comment

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

IT LGTM

dmmqz
dmmqz reviewed Aug 30, 2025
View reviewed changes
Copy link
Member

@dmmqz dmmqz left a comment

Choose a reason for hiding this comment

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

In the last example of the pt_BR page you can add long options for -k and -h.

@tldr-bot
Copy link

Hello! I've noticed something unusual when checking this PR:

  • The page pages.it/common/docker-images.md is outdated, based on number of commands, compared to the English page.
  • The page pages.pl/common/docker-images.md is outdated, based on number of commands, compared to the English page.
  • The page pages.tr/common/docker-images.md is outdated, based on number of commands, compared to the English page.

Is this intended? If so, just ignore this comment. Otherwise, please double-check the commits.

@nelsonfigueroa nelsonfigueroa merged commit f92cf52 into main Aug 30, 2025
8 checks passed
@nelsonfigueroa nelsonfigueroa deleted the docker-images-refresh branch August 30, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SpikeTheDragon40k SpikeTheDragon40k SpikeTheDragon40k approved these changes

@dmmqz dmmqz dmmqz approved these changes

@spageektti spageektti spageektti approved these changes

@TheRootDaemon TheRootDaemon TheRootDaemon approved these changes

@tansiret tansiret Awaiting requested review from tansiret tansiret is a code owner

@isaacvicente isaacvicente Awaiting requested review from isaacvicente isaacvicente is a code owner

@renie renie Awaiting requested review from renie renie is a code owner

@acuteenvy acuteenvy Awaiting requested review from acuteenvy acuteenvy is a code owner

@mebeim mebeim Awaiting requested review from mebeim mebeim is a code owner

@Magrid0 Magrid0 Awaiting requested review from Magrid0 Magrid0 is a code owner

Assignees
No one assigned
Labels
translation edit Changes to an existing translation page(s).
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@nelsonfigueroa @tldr-bot @SpikeTheDragon40k @dmmqz @spageektti @TheRootDaemon