Skip to content

docs: consistency audit across README, docs site, and zenoh-bridge demo#64

Closed
oguzkaganozt wants to merge 2 commits into
mainfrom
fix/docs-consistency-audit
Closed

docs: consistency audit across README, docs site, and zenoh-bridge demo#64
oguzkaganozt wants to merge 2 commits into
mainfrom
fix/docs-consistency-audit

Conversation

@oguzkaganozt
Copy link
Copy Markdown
Collaborator

Summary

  • Docs consistency pass: align terminology, deduplicate, and tighten instructions across the top-level README.md, deployment READMEs, and the MkDocs site under docs/.
  • Updates doc_env/Dockerfile, Makefile doc targets, and .github/workflows/deploy-docs.yaml to stay consistent with the doc build.
  • Zenoh-bridge demo fixes: standardize the "Open AD Kit" name (drop the "OpenADKit" / "Open Ad Kit" outliers), correct the architecture diagram port label from 6080 to 6081 (the host-mapped port users actually hit), and set REMOTE_PASSWORD=openadkit in deployments/demos/zenoh-bridge/docker-compose.yaml — previously truncated to o, which contradicted the documented default.
  • Small polish in docs/deployments/samples/index.md (added one-liner descriptions for the two samples, mirroring docs/deployments/index.md).

Test plan

  • mkdocs build (or the make doc target) completes without warnings about broken links or missing nav entries.
  • Render the zenoh-bridge page and confirm the architecture diagram now shows "HTTP (Port 6081)".
  • Bring up the zenoh-bridge demo (./cloud.sh up -d + ./edge.sh up -d) and verify http://localhost:6081 is reachable with password openadkit.
  • Skim the rendered samples index and confirm both Planning and Logging entries show descriptions.
  • Visually diff the rendered top-level README on GitHub and spot-check the reworded bullets (images/runtimes/components phrasing).

Align terminology and instructions across top-level README, deployment
READMEs, and docs pages; update doc_env Dockerfile, Makefile, and
deploy-docs workflow to match.
- Standardize "Open AD Kit" casing (drop "OpenADKit" / "Open Ad Kit" outliers).
- Correct zenoh-bridge architecture diagram port label to 6081 (host-mapped).
- Add descriptions to the samples deployments index for parity with parent.
- Set REMOTE_PASSWORD=openadkit in zenoh-bridge compose so it matches the
  documented default; previously it was truncated to "o".
@oguzkaganozt
Copy link
Copy Markdown
Collaborator Author

Please review @youtalk @mitsudome-r

@oguzkaganozt
Copy link
Copy Markdown
Collaborator Author

Superseded by three scoped follow-up PRs for easier review:\n\n- #70 (docs tooling/build consistency)\n- #71 (zenoh-bridge correctness fixes)\n- #72 (editorial docs consistency pass)\n\nClosing this PR in favor of those three.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant