Show a one-off warning about onlyAnalyzeProjectsWithOpenFiles#5755
Show a one-off warning about onlyAnalyzeProjectsWithOpenFiles#5755
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as resolved.
This comment was marked as resolved.
3364983 to
29cd824
Compare
|
@bwilkerson @jwren any opinions on the text for this notification? (it would be shown only to users with the setting enabled, and once they click the button to open the issue or dismiss the notification, we would not show it again).
The button opens dart-lang/sdk#61742 which has some details about why I think we should deprecate this (TL;DR, it makes initial analysis faster, but can cause long delays as we reset analysis roots as the first/last file in any project is opened/closed during normal use). |
|
That message looks good to me. If I were going to change anything I might suggest removing "you think", just because I don't think it adds to the meaning, but I'm also fine with leaving it in. |
29cd824 to
221c354
Compare
Yep, I agree - done. Thanks! |

See dart-lang/sdk#61742