Skip to content

test: add tests for remotePatterns#730

Merged
vicb merged 4 commits intomainfrom
vicb/rp-clean
Jun 16, 2025
Merged

test: add tests for remotePatterns#730
vicb merged 4 commits intomainfrom
vicb/rp-clean

Conversation

@vicb
Copy link
Copy Markdown
Contributor

@vicb vicb commented Jun 16, 2025

  • use the compiled patterns from the image manifest instead of re-computing them
  • add unit and integration tests for remote patterns
  • extract the code out of init to images
  • 404 -> 400 to conform with Next.js behavior

/cc @penalosa

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 16, 2025

🦋 Changeset detected

Latest commit: 1af96b6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@opennextjs/cloudflare Patch

Not sure what this means? Click here to learn what changesets are.

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

@vicb vicb requested review from conico974 and james-elicx and removed request for james-elicx June 16, 2025 08:14
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Jun 16, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@opennextjs/cloudflare@730

commit: 1af96b6

@vicb vicb requested a review from james-elicx June 16, 2025 12:40
Comment thread packages/cloudflare/src/cli/templates/images.ts
Comment thread packages/cloudflare/src/cli/templates/images.ts
@vicb
Copy link
Copy Markdown
Contributor Author

vicb commented Jun 16, 2025

Thanks for the review James!

Comment thread packages/cloudflare/src/cli/templates/images.ts
Comment thread packages/cloudflare/src/cli/templates/images.ts Outdated
@vicb vicb merged commit ae31733 into main Jun 16, 2025
7 checks passed
@vicb vicb deleted the vicb/rp-clean branch June 16, 2025 16:33
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