Skip to content
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

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

sgrekhov
Copy link
Contributor

@sgrekhov sgrekhov commented Mar 4, 2025

Again small formatting added. Removed or added empty lines

@eernstg
Copy link
Member

eernstg commented Mar 4, 2025

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 // Requirements=nnbd-strong, (4) some bug fixes (changing libFunc to libFunc() and such, in Parts-with-imports), (5) renaming enum-shorthands to dot-shorthands, and (6) moving @issue around in a few cases. I'm not sure, but it seems likely that some of those new/deleted cases would actually be renamings.

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.

@sgrekhov
Copy link
Contributor Author

sgrekhov commented Mar 5, 2025

@eernstg, how can you see that diff? It's a rather big diff, but it contains: (1) the removal of // Requirements=nnbd-strong, (2) the removal of excessive empty lines, (3) the addition of empty lines where needed, and (4) the removal of files containing // Requirements=nnbd-weak. There are no changes involving the addition of new libraries or the renaming of enum-shorthands to dot-shorthands. Are you looking at the diff shown when you click the 'Files changed' tab on this GitHub page?

@eernstg
Copy link
Member

eernstg commented Mar 5, 2025

With a PR that changes many files, I'm usually fetching it (here: gh pr checkout 3094) and using various command line tools to understand what's going on. This yields the 46707 line diff.

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 master, which introduced a bunch of spurious changes.

With the fetch, the diff is 3803 lines. ;-)

Copy link
Member

@eernstg eernstg left a 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! ;-)

@eernstg eernstg merged commit 5e29c78 into dart-lang:master Mar 5, 2025
2 checks passed
@sgrekhov
Copy link
Contributor Author

sgrekhov commented Mar 5, 2025

Thanx!

@eernstg
Copy link
Member

eernstg commented Mar 5, 2025

Thanks!

copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Mar 7, 2025
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants