Skip to content

Commit 7d64beb

Browse files
committed
Resolve the TODO in FlutterPubspecNotificationProvider, with an additional comment.
1 parent 33b9fa6 commit 7d64beb

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

src/io/flutter/editor/FlutterPubspecNotificationProvider.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,8 @@ public EditorNotificationPanel createNotificationPanel(@NotNull VirtualFile file
4949
}
5050

5151
// The Bazel workspace condition is handled by this check, pubspecs are not used.
52-
// TODO(jwren) Add additional check to exit if this is a Flutter Bazel workspace (in the event that some pubspec.yaml file is opened).
52+
// If for some reason a pubspec is opened, the next condition, isFlutterModule, will return false,
53+
// there is no reason to call FlutterModuleUtils.isFlutterBazelProject().
5354
if (!PubRoot.isPubspec(file)) {
5455
return null;
5556
}

0 commit comments

Comments
 (0)