Skip to content

Misc minor changes - #49

Merged
vicb merged 4 commits into
mainfrom
vicb/misc
Aug 28, 2026
Merged

Misc minor changes#49
vicb merged 4 commits into
mainfrom
vicb/misc

Conversation

@vicb

@vicb vicb commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

The first commit add instructions to run the tests locally.

The second commit remove the c3 template for Cloudflare. It is outdated and not maintained.
We'll add a new template when the adapters is ready.


Devin Review

@vicb
vicb requested review from conico974 and a lite review from Copilot August 28, 2026 07:52
@pkg-pr-new

pkg-pr-new Bot commented Aug 28, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/opennextjs/adapters-api/@opennextjs/aws@a415ed6
npm i https://pkg.pr.new/opennextjs/adapters-api/@opennextjs/cloudflare@a415ed6
npm i https://pkg.pr.new/opennextjs/adapters-api/@opennextjs/core@a415ed6

commit: a415ed6

@devin-ai-integration devin-ai-integration 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.

Devin Review found 1 potential issue.

Devin Review

Comment thread create-cloudflare/next/package.json

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

Pull request overview

Updates the repository documentation for running tests locally and removes an outdated/unmaintained Cloudflare “create” template from the monorepo.

Changes:

  • Expanded README.md testing instructions (unit + e2e, AWS + Cloudflare).
  • Removed the create-cloudflare/next template and its associated config/source/assets.

Reviewed changes

Copilot reviewed 17 out of 24 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
README.md Adds detailed local testing instructions (unit + e2e) and Cloudflare e2e steps.
create-cloudflare/next/wrangler.jsonc Removes outdated Cloudflare template Wrangler config.
create-cloudflare/next/tsconfig.json Removes template TypeScript config.
create-cloudflare/next/src/app/page.tsx Removes template app page.
create-cloudflare/next/src/app/layout.tsx Removes template root layout.
create-cloudflare/next/src/app/globals.css Removes template global styles.
create-cloudflare/next/README.md Removes template README.
create-cloudflare/next/public/window.svg Removes template asset.
create-cloudflare/next/public/next.svg Removes template asset.
create-cloudflare/next/public/globe.svg Removes template asset.
create-cloudflare/next/public/file.svg Removes template asset.
create-cloudflare/next/public/favicon.svg Removes template favicon.
create-cloudflare/next/public/_headers Removes template headers config.
create-cloudflare/next/postcss.config.mjs Removes template PostCSS config.
create-cloudflare/next/package.json Removes template package manifest.
create-cloudflare/next/open-next.config.ts Removes template OpenNext Cloudflare config.
create-cloudflare/next/next.config.ts Removes template Next.js config.
create-cloudflare/next/.vscode/settings.json Removes template editor settings.
create-cloudflare/next/.oxlintrc.json Removes template lint config.
create-cloudflare/next/.gitignore Removes template gitignore.
create-cloudflare/next/.dev.vars Removes template local Wrangler env vars.
Files not reviewed (1)
  • pnpm-lock.yaml: Generated file

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread README.md
Comment thread README.md
Comment thread README.md Outdated
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
@vicb vicb changed the title Misc minot changes Misc minor changes Aug 28, 2026
@vicb
vicb merged commit f89c157 into main Aug 28, 2026
8 checks passed
@vicb
vicb deleted the vicb/misc branch August 28, 2026 08:07
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