-
-
Notifications
You must be signed in to change notification settings - Fork 6
deps: zero v0.22.0 #100
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
base: main
Are you sure you want to change the base?
deps: zero v0.22.0 #100
Conversation
would love this to be merged and released 🙏 @danielroe edit: actually, let me review to be sure |
yes, my only question is about |
Yes, that's what I was thinking as well... @arv I read the PR adding Do you want to expose those? Or are you looking to merge |
I don't think it is worth exporting, it is easy enough to redefine. Vue bindings in zero npm package is still the plan. Unsure if it will be in beta or not. It is all a matter of prioritizing. |
It seems reasonable to export the ms version since we are already exporting the string version. |
This release changes how TTL works. Changes made:
DEFAULT_TTL_MS
since it's not exported anymore