Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

fix(docs): repair broken links in CONTRIBUTING.md#13647

Merged
alexrashed merged 1 commit into
localstack:mainfrom
Septic-H:fix/doc-links
Jan 27, 2026
Merged

fix(docs): repair broken links in CONTRIBUTING.md#13647
alexrashed merged 1 commit into
localstack:mainfrom
Septic-H:fix/doc-links

Conversation

@Septic-H

Copy link
Copy Markdown
Contributor

Motivation

While setting up my development environment to contribute to LocalStack, I accessed the Contributing guidelines via the repository's main page overlay. I noticed that clicking on key setup links (specifically "Development Environment Setup", "LocalStack Concepts", and"Testing") resulted in 404 Page Not Found errors. Spent a solid 5 mins wondering where the files are supposed to be.
This happens because the overlay renders the Markdown from the repository root, breaking the relative links that assume the user is browsing inside the docs/ directory.
Screenshot from 2026-01-26 21-00-20

Changes

Updated the links in docs/CONTRIBUTING.md to use root-relative paths (e.g., /docs/development-environment-setup/README.md). This ensures the links resolve correctly in both the file browser view and the main "Contributing" overlay tab.

Tests

Manually verified that the new paths correctly point to the existing files in the repository structure.

Related

N/A

Update relative links to use root-relative paths (/docs/...) so they resolve correctly when viewing the file via the GitHub "Contributing" overlay tab.

@alexrashed alexrashed 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.

Hi @Septic-H!
Could you maybe share concrete links on how to find the currently deployed wrong links?
I tried going to https://github.com/localstack/localstack/contribute and then to "Read the contributing guidelines" and then click through the first few prominent links. They all worked just fine for me?

@alexrashed alexrashed added semver: patch Non-breaking changes which can be included in patch releases docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes labels Jan 27, 2026
@alexrashed alexrashed added this to the Playground milestone Jan 27, 2026
@Septic-H

Copy link
Copy Markdown
Contributor Author

Hey @alexrashed!
Yea sure. The issue specifically happens when viewing the "Contributing" overlay (which renders from root), rather than viewing the file directly in the docs folder.

You can reproduce it instantly with this link:
https://github.com/localstack/localstack?tab=contributing-ov-file

Or, to navigate there manually:

  1. On the main page (https://github.com/localstack/localstack), scroll down to the README.
Screenshot from 2026-01-27 18-13-29
  1. Click on "Contributing".
image
  1. Then click on any of these following links (except "Contribution Guidelines"):
image

It gives a 404 error because it tries to find the file relative to root, not docs/.
image

@alexrashed alexrashed 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.

That is a great catch, thank you very much for the proactive contribution, and for the explanations on how to reproduce it!
I'll directly merge this one. 🚀

@alexrashed alexrashed merged commit 44d1022 into localstack:main Jan 27, 2026
16 of 19 checks passed
@Septic-H Septic-H deleted the fix/doc-links branch January 27, 2026 13:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

docs: skip Pull request does not require documentation changes notes: skip Pull request does not have to be mentioned in the release notes semver: patch Non-breaking changes which can be included in patch releases

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants