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.

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

nyannyacha
Copy link
Contributor

What kind of change does this PR introduce?

Enhancement

Description

...WIP

@nyannyacha
Copy link
Contributor Author

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 linked an issue Sep 15, 2024 that may be closed by this pull request
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

@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.

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.

InvalidWorkerCreation: Edge functions cannot handle concurrent requests
2 participants