Skip to content

docs: add cloudflare deploy guide - #37

Open
losolio wants to merge 1 commit into
mainfrom
docs/cloudflare-deploy
Open

docs: add cloudflare deploy guide#37
losolio wants to merge 1 commit into
mainfrom
docs/cloudflare-deploy

Conversation

@losolio

@losolio losolio commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings August 2, 2026 11:03

Copilot AI 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.

Pull request overview

Adds a dedicated Cloudflare deployment guide to the documentation, and links to it from the CLI docs so readers can follow a Cloudflare-specific walkthrough after generating dist/.

Changes:

  • Add docs/deployment/cloudflare.md with Cloudflare Pages + Wrangler instructions, CI notes, and a Workers alternative.
  • Link to the new Cloudflare deployment guide from docs/cli.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
docs/deployment/cloudflare.md New Cloudflare deployment walkthrough (Pages, CI env vars, 404 fallback, Workers alternative).
docs/cli.md Adds a pointer to the Cloudflare deployment guide from the “Build and deploy” section.
Suppressed comments (2)

docs/deployment/cloudflare.md:16

  • The “fast path” snippet is missing the build step; wrangler pages deploy dist will fail if dist/ hasn’t been generated yet. Adding npx lectio-docs build makes the snippet copy/pasteable.
npx wrangler login                                    # first time only
npx wrangler pages deploy dist --project-name my-docs

docs/deployment/cloudflare.md:64

  • The wrangler.jsonc example omits main and an assets binding; Wrangler configs typically require main, and a binding is needed if the Worker is responsible for serving assets. As written, readers may get a config that fails validation or can’t serve dist/ as intended.
{
  "name": "my-docs",
  "compatibility_date": "2026-08-02",
  "assets": { "directory": "./dist", "not_found_handling": "404-page" }
}

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +8 to +10
`lectio build` leaves a fully static site in `./dist` — every page prerendered
to HTML, with the search index at `/search-index.json`. Nothing runs
server-side, so Cloudflare can serve it as plain static assets.
Comment on lines +55 to +56
Cloudflare now steers new projects toward Workers with static assets. It's the
same static `dist/`, plus a small config at the repo root:
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.

2 participants