Closed davidkpiano closed 2 months ago
Latest commit: 996b511b4e410897800c2582940d128790e351a8
The changes in this PR will be included in the next version bump.
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
@tonivj5 Do you have any thoughts on this to share? (responding to emoji reaction)
@Andarist Types are failing
hey @davidkpiano! My "corcerns" (or doubts) about these changes were about make zod "first citizen", I love the idea itself, but I personally prefer to avoid "hard" coupling and let it open to integrate other solutions using adapters or some other kind of technique (and I think @colinhacks is drafting an awesome solution to achive it with https://github.com/standard-schema/standard-schema).
Zod is a brilliant piece of software, I've used it before although I love and use other libraries too as https://github.com/sinclairzx81/typebox and I think this field is a moving target that it's going to beneficate from making it elegible.
Anyway, I'm just passing through here so take my reactions and comments with a grain of salt 😅. I love from time to time review your work and check what's cooking here 😸
@tonivj5 You're in luck - we moved the Zod stuff out and that will probably be a separate PR later, if we decide to do it.
You can now emit events from a store:
You can make emitted events type-safe via
createStore({ types: { … } })