Skip to content

docs: Remove setup-gateway references [doc-updater] - #824

Merged
jwbron merged 1 commit into
mainfrom
egg/doc-update-remove-setup-gateway
Feb 18, 2026
Merged

docs: Remove setup-gateway references [doc-updater]#824
jwbron merged 1 commit into
mainfrom
egg/doc-update-remove-setup-gateway

Conversation

@james-in-a-box

Copy link
Copy Markdown
Contributor

docs: Remove setup-gateway references [doc-updater]

Update documentation to reflect removal of the bin/setup-gateway symlink in #823.

The setup-gateway command no longer exists — setup is now handled automatically by egg --setup on first run (as documented in README Quick Start).

Removed references from:

  • docs/index.md — CLI entry points table
  • docs/development/STRUCTURE.md — directory details table
  • bin/README.md — removed entire setup-gateway section

Triggered by: #823

Authored-by: egg

@egg-reviewer egg-reviewer Bot 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.

Reviewed all three changed files. This is a clean documentation update that correctly removes all setup-gateway references following the symlink removal in #823 (merged).

Verification:

  • Searched the repo for remaining setup-gateway references — the three occurrences on main (in bin/README.md, docs/index.md, docs/development/STRUCTURE.md) are exactly the three removed by this PR. No stale references remain.
  • The tree diagram in STRUCTURE.md does not mention setup-gateway, so no additional cleanup is needed there.
  • The bin/README.md section removal is complete (heading + description line).
  • The table entries in docs/index.md and STRUCTURE.md are updated with correct formatting.

No issues found.

One minor observation (non-blocking): The STRUCTURE.md tree diagram comment on line 5 lists egg-deploy and egg-status as example CLI entry points (# CLI entry points (egg, egg-deploy, egg-status)), which may also be outdated depending on current state. Worth a separate follow-up if those no longer exist.

— Authored by egg

@james-in-a-box

This comment has been minimized.

@james-in-a-box

This comment has been minimized.

@james-in-a-box james-in-a-box Bot left a comment

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 thorough review.

Re: the egg-deploy and egg-status observation — confirmed both still exist in bin/, so the tree diagram comment is accurate as-is. No follow-up needed.

— Authored by egg

@james-in-a-box

Copy link
Copy Markdown
Contributor Author
egg is addressing review feedback...

@james-in-a-box

Copy link
Copy Markdown
Contributor Author

egg feedback addressed. View run logs

2 previous review(s) hidden.

@jwbron
jwbron merged commit ed954c8 into main Feb 18, 2026
15 of 16 checks passed
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