Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(cloudflare+cloudflare-pages+cloudflare-workers+templates): add support for @cloudflare/workers-types v3 #645

Closed
wants to merge 1 commit into from

Conversation

edmundhung
Copy link
Contributor

@edmundhung edmundhung commented Nov 25, 2021

Cloudflare released an update for @cloudflare/workers-types last month. It is now automatically generated from the source as announced here and this ensures typing to be always up-to-dated with the latest runtime.

However, the current CF adapter has a peerDependency of v2 only. It would be nice if we could add support of v3 as well.
This will also makes experimenting with new ESM worker easier (as all new types are only defined on v3) before official support is ready.

Thanks for all the great work!

@MichaelDeBoey
Copy link
Member

MichaelDeBoey commented Mar 28, 2022

@edmundhung Could you please rebase your branch onto latest dev & fix conflicts?

Also don't forget to update the remix-cloudflare & remix-cloudflare-pages packages, as that's a fairly new package

@MichaelDeBoey MichaelDeBoey added the needs-response We need a response from the original author about this issue/PR label Mar 28, 2022
@jensmeindertsma
Copy link

Yeah this would be great, thanks Edmund 😊

@edmundhung
Copy link
Contributor Author

edmundhung commented Mar 28, 2022

@edmundhung Could you please rebase your branch onto latest dev & fix conflicts?

Also don't forget to update the remix-cloudflare & remix-cloudflare-pages packages, as that's a fairly new package

@MichaelDeBoey I have opened a new PR #2530 as I accidentally deleted the original fork repository and now I cannot push any commit to this PR anymore. (Edit: Feel free to close this PR after you check the message)

@github-actions github-actions bot removed the needs-response We need a response from the original author about this issue/PR label Mar 28, 2022
@MichaelDeBoey
Copy link
Member

Closing in favour of #2530

@MichaelDeBoey MichaelDeBoey added the duplicate This issue or pull request already exists label Mar 28, 2022
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.

4 participants