-
-
Notifications
You must be signed in to change notification settings - Fork 8.5k
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
types: improve readability of built-in type #9129
Conversation
Date
typesDate
type
Size ReportBundles
Usages
|
Date
type0462f86
to
c2b674f
Compare
I deleted |
CodSpeed Performance ReportMerging #9129 will improve performances by 63.31%Comparing Summary
Benchmarks breakdown
|
I think you misunderstood what I meant - That said my last requested change seems unnecessary, since |
Understood! I will revert back to the previous commit. |
This reverts commit 57dc127.
# Conflicts: # packages/reactivity/src/ref.ts
Description
We could write this in code, but the types would be unwrapped:
We can declare types that should avoid reference unwrapping by writing:
But
Date
is a commonly used built-in type, would it be better if we support it directly?Before
After