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
Revert "Refactor JSNumber.toDart and Object.toJS" #43286
Conversation
This reverts commit ca41476. Reason for reverting: Broke the roll into the framework.
See more context about the revert: https://discord.com/channels/608014603317936148/1123298130671194173 |
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!
auto label is removed for flutter/engine, pr: 43286, due to - The status or check suite Linux linux_clang_tidy has failed. Please fix the issues identified (or deflake) before re-applying this label. |
A reverts link could not be found or was formatted incorrectly. Format is 'Reverts owner/repo#id' |
…129682) flutter/engine@7c7c45d...25a5850 2023-06-27 58529443+srujzs@users.noreply.github.com Revert "Refactor JSNumber.toDart and Object.toJS" (flutter/engine#43286) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Please CC jimgraham@google.com,rmistry@google.com,zra@google.com on the revert to ensure that a human is aware of the problem. To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Reverts flutter#43149 Reason for reverting: Broke the roll into the framework.
Reverts #43149
Reason for reverting: Broke the roll into the framework.
Pre-launch Checklist
///
).