-
-
Notifications
You must be signed in to change notification settings - Fork 442
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
refactor(deserialize): allow setting name for root deserialized value #1463
Conversation
✅ Deploy Preview for biomejs canceled.
|
Parser conformance results onjs/262
jsx/babel
symbols/microsoft
ts/babel
ts/microsoft
|
6d380c4
to
ca0b24d
Compare
ca0b24d
to
aaad110
Compare
CodSpeed Performance ReportMerging #1463 will degrade performances by 15.87%Comparing Summary
Benchmarks breakdown
|
aaad110
to
b7c45e1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a test where we use this new field?
You mean in the doctest? |
b7c45e1
to
fc6b8dd
Compare
e744f99
to
2257da3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lovely, thank you!
Summary
Allow naming the root deserialized value.
This addresses some requests of #1250.