fix: wrangler config not passed to cache population

[james-elicx]

fixes #813

[changeset-bot]

🦋 Changeset detected

Latest commit: b5c74fa

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

[pkg-pr-new]

Open in StackBlitz

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

commit: b5c74fa

[vicb]

The changes look good, thanks for the fix!

We should get this in when the merge glitch is fixed.

There are 2 things we should look at for the CLI:

• this comment
• unify how we pass the env and config path to runWrangler (sometime via wranglerOpts, sometimes directly in the args)

[vicb]

Why do we --skipWranglerConfigCheck here?
Shouldn't we --config wrangler.e2e.jsonc instead ?

[james-elicx]

The build doesn't need to know about the config, it just has a check at the end of running to see if one exists and prompts to let you create one if not

[vicb]

I was thinking about initOpenNextCloudflareForDev(); but we can wait until we tackle this in a follow up PR

[james-elicx]

There are 2 things we should look at for the CLI:

• this comment
• unify how we pass the env and config path to runWrangler (sometime via wranglerOpts, sometimes directly in the args)

Agree we should look at both of those.

[vicb]

Thanks!