-
Notifications
You must be signed in to change notification settings - Fork 28
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
#3088. Remove weak mode from LanguageFeatures/nnbd. Part 2. #3094
Conversation
It's a 46707 line diff, and it seems to include (1) a rather large number of completely new libraries, (2) a somewhat similar number of deleted libraries, as well as (3) removal of It would be great if the diff could be made less massive and more homogeneous (hence: easier to read) by separating those elements into a handful of simpler PRs. |
@eernstg, how can you see that diff? It's a rather big diff, but it contains: (1) the removal of |
With a PR that changes many files, I'm usually fetching it (here: Ah! Now I can see what happened: We just had a lot of hardware network changes during the weekend so I had to start using a cloudtop because there was no access to my usual desktop. So I was working from a backup on the cloudtop, and the co19 repo wasn't up to date (even though I had just fetched all updates when I had access to the desktop for a short period of time). So I compared this branch with a not-quite new With the |
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, when looking carefully! ;-)
Thanx! |
Thanks! |
2025-03-06 [email protected] dart-lang/co19#3057. Add reachability tests for operators. Part 3. (dart-lang/co19#3100) 2025-03-06 [email protected] dart-lang/co19#3057. Add reachability tests for operators. Part 2. (dart-lang/co19#3099) 2025-03-06 [email protected] dart-lang/co19#3057. Add reachability tests for operators. Part 1. (dart-lang/co19#3098) 2025-03-05 [email protected] dart-lang/co19#3057. Add reachability tests for operator + (dart-lang/co19#3097) 2025-03-05 [email protected] dart-lang/co19#3057. Add tests for cascade method invocation (dart-lang/co19#3095) 2025-03-05 [email protected] Closes dart-lang/co19#3088. Remove weak mode from LanguageFeatures/Patterns. (dart-lang/co19#3096) 2025-03-05 [email protected] dart-lang/co19#3088. Remove weak mode from LanguageFeatures/nnbd. Part 2. (dart-lang/co19#3094) 2025-03-04 [email protected] dart-lang/co19#3057. Add tests for method invocation (dart-lang/co19#3083) 2025-03-04 [email protected] dart-lang/co19#3088. Remove weak mode from LanguageFeatures/nnbd. Part 1. (dart-lang/co19#3092) 2025-03-04 [email protected] dart-lang/co19#3088. Remove weak mode from Language and LibTest (dart-lang/co19#3093) 2025-03-03 [email protected] dart-lang/co19#3088. Remove weak mode from TypeSystem/subtyping (dart-lang/co19#3091) 2025-03-03 [email protected] dart-lang/co19#3088. Remove weak mode from TypeSystem/type-normalization (dart-lang/co19#3089) 2025-03-03 [email protected] dart-lang/co19#3088. Remove weak mode from TypeSystem/flow-analysis (dart-lang/co19#3090) 2025-03-03 [email protected] Fixes dart-lang/co19#3086. Update context type for expected value (dart-lang/co19#3087) Cq-Include-Trybots: luci.dart.try:analyzer-linux-release-try Change-Id: Ib3b8714c4d6ae5d73089b41374ed3ce16b6ddab0 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/414220 Reviewed-by: Erik Ernst <[email protected]> Reviewed-by: Alexander Thomas <[email protected]> Commit-Queue: Alexander Thomas <[email protected]>
Again small formatting added. Removed or added empty lines