Skip to content

Clarify Pages C3 getting started guide#31119

Draft
dario-piotrowicz wants to merge 1 commit into
productionfrom
dario/DEVX-2432/c3-pages-frameworks
Draft

Clarify Pages C3 getting started guide#31119
dario-piotrowicz wants to merge 1 commit into
productionfrom
dario/DEVX-2432/c3-pages-frameworks

Conversation

@dario-piotrowicz
Copy link
Copy Markdown
Member

Summary

Screenshots (optional)

Documentation checklist

  • Is there a changelog entry (guidelines)? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to RSS feeds, the Discord, and X.
  • The change adheres to the documentation style guide.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions
Copy link
Copy Markdown
Contributor

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
/src/content/docs/pages/ @cloudflare/workers-docs, @GregBrimble, @WalshyDev, @aninibread, @irvinebroque, @cloudflare/product-owners, @MattieTK

Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

2 minor wording suggestions and 1 PR hygiene issue.

- `--category` <Type text="string" /> <MetaInfo text="optional" />

- The kind of templates that should be created.
- The kind of templates that should be created. When `--platform=pages` is specified, only `web-framework` and `remote-template` are available since other categories only produce Workers projects.
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.

Avoid repeating 'only' in the same sentence:

Suggested change
- The kind of templates that should be created. When `--platform=pages` is specified, only `web-framework` and `remote-template` are available since other categories only produce Workers projects.
- The kind of templates that should be created. When `--platform=pages` is specified, `web-framework` and `remote-template` are the only available categories, as the others produce Workers projects.

- `--framework` <Type text="string" /> <MetaInfo text="optional" />

- The type of framework to use to create a web application (when using this option, `--type` is ignored).
- The type of framework to use to create a web application (when using this option, `--type` is ignored). When `--platform=pages` is specified, only frameworks that support Pages are shown.
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.

Use active voice for consistency:

Suggested change
- The type of framework to use to create a web application (when using this option, `--type` is ignored). When `--platform=pages` is specified, only frameworks that support Pages are shown.
- The type of framework to use to create a web application (when using this option, `--type` is ignored). When `--platform=pages` is specified, C3 shows only frameworks that support Pages.

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 28, 2026

Review complete — 2 inline suggestions posted.

Summary

The PR updates the Pages C3 guide to clarify that only a subset of frameworks support Pages deployment via C3, removing Next.js, Remix, SolidStart, and Redwood from the Pages-specific list and marking them as Workers-only in the CLI reference. The typo fix ("you" → "your") and the added note about non-framework paths are good improvements.

Issues

MEDIUM — PR body is empty
The pull request description contains only the default template with no actual summary. A maintainer (or the author) should add context explaining why these frameworks are no longer listed for Pages via C3, so reviewers understand the source-of-truth for the change.

LOW — Repetitive wording (line 95)
The --category description uses "only" twice in one sentence. Suggested rephrase posted inline.

LOW — Passive voice (line 120)
The --framework description uses passive voice ("...only frameworks that support Pages are shown"). Suggested active-voice rephrase posted inline.

Labels

Already applied: product:pages, size/s — both appropriate. No additional labels needed.

github run

@github-actions
Copy link
Copy Markdown
Contributor

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants