-
Notifications
You must be signed in to change notification settings - Fork 224
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
Switch suggest component from UDL to proc-macros #6380
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6380 +/- ##
=======================================
Coverage 52.72% 52.72%
=======================================
Files 125 125
Lines 12820 12820
=======================================
Hits 6759 6759
Misses 6061 6061 ☔ View full report in Codecov by Sentry. |
a085e47
to
84f7b04
Compare
megazords/ios-rust/MozillaTestServices/MozillaTestServices.xcodeproj/project.pbxproj
Show resolved
Hide resolved
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.
Looks great. To me this seems perfect, but we should wait for @linabutler to confirm that project.pbxproj
change.
megazords/ios-rust/MozillaTestServices/MozillaTestServices.xcodeproj/project.pbxproj
Show resolved
Hide resolved
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.
So cool! This looks just wonderful to me, too; thanks!
Deleting the
suggest.udl
file and use proc-macros to export directly from the Rust codebase.