-
Notifications
You must be signed in to change notification settings - Fork 104
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: WebIDL derive macro #1003
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1003 +/- ##
==========================================
+ Coverage 81.43% 81.76% +0.32%
==========================================
Files 97 103 +6
Lines 23877 27657 +3780
==========================================
+ Hits 19445 22613 +3168
- Misses 4432 5044 +612 ☔ View full report in Codecov by Sentry. |
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.
LGTM! Let's try it out in Deno and iterate based on what's needed.
cc @ry @bartlomieju
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.
LGTM too, let's try it out.
WebIdlConverter
traitUnrestricted
structs for unrestricted float and unrestricted doubleByteString
struct for bytestring conversionBigInt
struct for bigint conversionWebIDL
derive macro, current only allowed on structs as a dictionary converter and on enums as enum converters