-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Support 2.0 in the analyzer independent of the new front end #31640
Comments
@MichaelRFairhurst, I added a line above for |
Updated the first line item with a resolution re: using flags to enable these features. |
…alyzer_cli and migrate the old behavior of --preview-dart-2 to --use-cfe. The --preview-dart-2 flag is now a no-op, with functionality coming soon. See #31640 Change-Id: I2db2e7ef6a324cbd5d369b807e331495fed167df Reviewed-on: https://dart-review.googlesource.com/33121 Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Jaime Wren <[email protected]>
Updated to show that #30177 is closed |
I believe that this issue is now obsolete. Please re-open if that's not true. |
The checkbox for #31638 is not yet checked. That bug is incomplete because its checkbox for #32156 is not yet checked. #32156 is still being worked on by @jmesserly - see https://dart-review.googlesource.com/c/sdk/+/45141. I'm going to ping that CL in the hopes that it can be landed soon, and I'll adjust #32156's priority to P1 to match this bug. |
All remaining unfinished subtasks are now being tracked in #32395. |
This is a tracking issue for supporting the Dart 2.0 semantics in analyzer without using the new front end.
update: we'll use
--preview-dart-2
to mean 'enable Dart 2.0 syntax', and use--use-cfe
to mean use the CFE--use-cfe
flag to replace current uses of--preview-dart-2
. @jwrenMoved to post alpha2:
The text was updated successfully, but these errors were encountered: