-
-
Notifications
You must be signed in to change notification settings - Fork 312
feat: Function Parameters for Effect.Service #4911
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: Function Parameters for Effect.Service #4911
Conversation
🦋 Changeset detectedLatest commit: 6cc9656 The changes in this PR will be included in the next version bump. This PR includes changesets to release 32 packages
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 |
I would not try to do memoization, that should be left to the user if they want to. At the moment the rule for functions returning layers is that to have a stable instance you need to extract the function call into a stable reference, which is what we do for anything like |
aa2ec73
to
c051853
Compare
b8850b4
to
c6bd1f8
Compare
Thanks for making this actually functional @tim-smart! |
07444af
to
2212e0f
Compare
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Co-authored-by: Tim <[email protected]>
Type
Description
POC to adding overloads to all Effect.Service's
effect
andscoped
variants to accept function parametersRelated