Skip to content

docs: fix broken docker docs links in CONTRIBUTING.md#52228

Merged
thaJeztah merged 1 commit into
moby:masterfrom
highloadninja:update-contributing-links
Mar 25, 2026
Merged

docs: fix broken docker docs links in CONTRIBUTING.md#52228
thaJeztah merged 1 commit into
moby:masterfrom
highloadninja:update-contributing-links

Conversation

@highloadninja

@highloadninja highloadninja commented Mar 25, 2026

Copy link
Copy Markdown
Contributor

- What I did
Fixed broken (404) links in CONTRIBUTING.md.

- How I did it
Replaced outdated docs.docker.com URLs with the current valid documentation URLs.

- How to verify it
Check the changed lines in CONTRIBUTING.md and click the updated links to ensure they resolve correctly to the active documentation pages.

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)
🐈

Comment thread CONTRIBUTING.md Outdated
quickly. If your pull request is not accepted on the first try,
don't get discouraged! Our contributor's guide explains [the review process we
use for simple changes](https://docs.docker.com/contribute/overview/).
use for simple changes](https://github.com/moby/moby/blob/master/project/REVIEWING.md).

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Perhaps link to both https://github.com/moby/moby/blob/master/docs/contributing/README.md (which I think was the previous link) and reviewing process?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for the suggestion, @thaJeztah! That makes perfect sense. I've updated the sentence to include relative links to both the contributor's guide and the reviewing process.

Signed-off-by: Maksim Nikolaev <nmaks2012@yandex.ru>
@highloadninja highloadninja force-pushed the update-contributing-links branch from 8397109 to 91cd097 Compare March 25, 2026 18:30

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah added this to the 29.3.2 milestone Mar 25, 2026
@thaJeztah thaJeztah merged commit 669fb7e into moby:master Mar 25, 2026
206 of 209 checks passed
kovan added a commit to kovan/moby that referenced this pull request Mar 26, 2026
Replace two dead docs.docker.com links with correct in-repo paths:

- CONTRIBUTING.md: image link pointed to removed
  docs.docker.com/opensource/project/who-written-for/ page; now
  links to docs/contributing/who-written-for.md
- .github/PULL_REQUEST_TEMPLATE.md: contributing guide link pointed
  to removed docs.docker.com/opensource/code/ page; now links to
  docs/contributing/ on GitHub

Follow-up to moby#52228 which fixed most broken links but missed these two.

Fixes moby#45024

Signed-off-by: Kit Dallege <xaum.io@gmail.com>
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.

2 participants