Skip to content

Commit 5b33bad

Browse files
stereotype441commit-bot@chromium.org
authored andcommitted
Migration: fix trial_migration.dart's handling of required
Now that the migration tool inserts `required` rather than `@required` for required parameters, we need the trial_migration tool to understand the new behavior. Change-Id: I401310930120cf6e57af7ab633983b44415d404f Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/119081 Reviewed-by: Samuel Rawlins <[email protected]> Commit-Queue: Paul Berry <[email protected]>
1 parent 327bc45 commit 5b33bad

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

pkg/nnbd_migration/tool/trial_migration.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ class _Listener implements NullabilityMigrationListener {
107107
} else if (edit.replacement == "import 'package:meta/meta.dart';\n" &&
108108
edit.length == 0) {
109109
++numMetaImportsAdded;
110-
} else if (edit.replacement == '@required ' && edit.length == 0) {
110+
} else if (edit.replacement == 'required ' && edit.length == 0) {
111111
++numRequiredAnnotationsAdded;
112112
} else if ((edit.replacement == '/* ' ||
113113
edit.replacement == ' /*' ||

0 commit comments

Comments
 (0)