[CP] merge into [beta] channel: disallows imports to dart:mirrors on the web #42715
Labels
cherry-pick-approved
Label for approved cherrypick request
cherry-pick-review
Issue that need cherry pick triage to approve
merge-to-beta
Uh oh!
There was an error while loading. Please reload this page.
commit(s) to merge: c632a62
merge instructions: clean merge
what's the problem:
In 2.8 there is a static error on web compilers (dart2js and ddc) when apps import
dart:mirrors
. This was unintended (bug #42475).In 2.9-beta this error is gone. We debated removing the error in 2.8 (#42553) and decided to embrace the breaking change instead (#42714) and readd the error to 2.9 (c632a62)
why cherry-pick:
We want to avoid being inconsistent: no error in 2.7, error in 2.8, no error in 2.9, error in 2.10. To prevent that, we'd like to cherry-pick this change to 2.9-beta and eventually 2.9 stable.
Note: since 2.8 already has this breaking change, this cherry-pick is effectively only improving the error message, it shouldn't affect anything that the 2.8 breaking change didn't before.
/cc @kevmoo @mit-mit @whesse @athomas @vsmenon @franklinyow
The text was updated successfully, but these errors were encountered: