Skip to content

docs(sites): add deploy example to the root command - #138

Merged
jamie-at-bunny merged 1 commit into
mainfrom
docs/sites-deploy-example
Jul 31, 2026
Merged

docs(sites): add deploy example to the root command#138
jamie-at-bunny merged 1 commit into
mainfrom
docs/sites-deploy-example

Conversation

@jamie-at-bunny

Copy link
Copy Markdown
Member

No description provided.

@changeset-bot

changeset-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1ebee71

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@jamie-at-bunny
jamie-at-bunny marked this pull request as ready for review July 31, 2026 10:13
@bogdan-at-bunny

Copy link
Copy Markdown

@codex review

@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown

Greptile Summary

Adds bunny sites deploy as a static-site deployment example on the root command screen and removes trailing whitespace from the ASCII banner.

Confidence Score: 5/5

The PR appears safe to merge with no actionable defects identified.

The new entry only extends the root command’s static examples with an existing sites deployment command, while the banner edits remove trailing whitespace without changing displayed content.

Important Files Changed

Filename Overview
packages/cli/src/cli.ts Adds a valid static-site deployment example to the root landing page and performs non-functional whitespace cleanup.

Reviews (1): Last reviewed commit: "docs(sites): add deploy example to the r..." | Re-trigger Greptile

@jamie-at-bunny
jamie-at-bunny merged commit acdddf5 into main Jul 31, 2026
2 checks passed
@jamie-at-bunny
jamie-at-bunny deleted the docs/sites-deploy-example branch July 31, 2026 10:15

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1ebee71c56

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread packages/cli/src/cli.ts
["Create an edge script", "bunny scripts init"],
["Add a domain to manage DNS", "bunny dns zones add example.com"],
["Create a dev sandbox", "bunny sandbox create my-sandbox"],
["Deploy a static site", "bunny sites deploy"],

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep experimental Sites off the landing page

When users run bunny without arguments, this row now advertises sites deploy even though sitesNamespace remains in experimentalCommands and is still defined as a hidden namespace. This exposes an experimental feature on the public landing page while it remains absent from normal help, creating an inconsistent release surface; remove the example until Sites is fully promoted.

AGENTS.md reference: AGENTS.md:L1107-L1107

Useful? React with 👍 / 👎.

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.

3 participants