Skip to content

docs: fix LocalStack port, Compose configs key, and a doubled link slash#25309

Merged
craig-osterhout merged 1 commit into
docker:mainfrom
emmayusufu:docs/fix-broken-examples-localstack-pgadmin-bun
Jun 10, 2026
Merged

docs: fix LocalStack port, Compose configs key, and a doubled link slash#25309
craig-osterhout merged 1 commit into
docker:mainfrom
emmayusufu:docs/fix-broken-examples-localstack-pgadmin-bun

Conversation

@emmayusufu

Copy link
Copy Markdown
Contributor

Three small fixes in the guides.

  • guides/localstack: the prose says S3_ENDPOINT_URL is http://localhost:4556, but LocalStack's edge port is 4566, and the guide's own .env block later uses 4566.
  • guides/pgadmin: the top-level Compose key for the pgpass file is configs, not config. As written the compose file is invalid.
  • guides/bun/deploy: the "Turn on Kubernetes" link has a doubled slash (/manuals//desktop/...).

- guides/localstack: prose S3_ENDPOINT_URL is `http://localhost:4556`, but
  LocalStack's edge port (and the guide's own .env) is 4566.
- guides/pgadmin: the top-level Compose key is `configs`, not `config`.
- guides/bun/deploy: doubled slash in the Kubernetes link path.

Signed-off-by: Emmanuel Yusufu Kimaswa <kimaswaemma36@gmail.com>
@netlify

netlify Bot commented Jun 9, 2026

Copy link
Copy Markdown

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 567c495
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6a279b9755cc6300087bf2fe
😎 Deploy Preview https://deploy-preview-25309--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.

@craig-osterhout craig-osterhout added the status/review Pull requests that are ready for review label Jun 10, 2026
@craig-osterhout

Copy link
Copy Markdown
Contributor

/review

@craig-osterhout craig-osterhout 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.

thank you. lgtm

@craig-osterhout craig-osterhout merged commit caa2754 into docker:main Jun 10, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/guides status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants