Skip to content

feat: expose useful props to Worker to be able to reuse the workers in js land #382

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 6 commits into
base: v1
Choose a base branch
from
Open

feat: expose useful props to Worker to be able to reuse the workers in js land #382

wants to merge 6 commits into from

Conversation

@nyannyacha
Copy link
Contributor

@nyannyacha nyannyacha commented Jul 4, 2024

What kind of change does this PR introduce?

Enhancement

Description

...WIP

@nyannyacha
Copy link
Contributor Author

nyannyacha commented Jul 4, 2024

Note:

Initial implementation has been completed.

@nyannyacha nyannyacha force-pushed the feat-reuse-worker-in-js-land branch from 71f944a to e1c870f Compare September 4, 2024 01:28
@nyannyacha nyannyacha mentioned this pull request Sep 15, 2024
Open
2 tasks
@nyannyacha nyannyacha linked an issue Sep 15, 2024 that may be closed by this pull request
InvalidWorkerCreation: Edge functions cannot handle concurrent requests #408
Open
2 tasks
@nyannyacha nyannyacha force-pushed the feat-reuse-worker-in-js-land branch from e1c870f to e1a025c Compare October 1, 2024 05:35
@nyannyacha nyannyacha force-pushed the feat-reuse-worker-in-js-land branch 6 times, most recently from b7dfda4 to de47fde Compare November 25, 2024 02:13
@nyannyacha nyannyacha force-pushed the feat-reuse-worker-in-js-land branch from de47fde to 47a63e2 Compare January 2, 2025 05:45
@nyannyacha nyannyacha force-pushed the feat-reuse-worker-in-js-land branch from 47a63e2 to 0f3f80c Compare January 5, 2025 21:59
@laktek laktek marked this pull request as ready for review January 7, 2025 04:50
@fbatiga
Copy link

fbatiga commented May 21, 2025

@nyannyacha what happened to this MR ?

@nyannyacha
Copy link
Contributor Author

nyannyacha commented May 21, 2025

@fbatiga Well... This PR has already been completed (Although there is no description yet 😅) and can be merged into the latest branch with a few minor modifications.

But it seems like other PRs have taken priority, so attempts to merge this one have been pushed back quite a bit.
The time will come soon when we can merge the pending PRs, so I think we should discuss merging this one then.

@fbatiga
Copy link

fbatiga commented Jul 2, 2025

@nyannyacha any news ? I'm currently contemplating migrating my edge functions to cloudflare workers.
because my ci/cd pipeline does not work...

@nyannyacha
Copy link
Contributor Author

nyannyacha commented Sep 9, 2025

@fbatiga I don't think this PR is related to the CI/CD problem.
Could you please open an issue with the reproduction steps if you are still using it?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

InvalidWorkerCreation: Edge functions cannot handle concurrent requests

2 participants

@nyannyacha @fbatiga