Skip to content

Conversation

@danrubel
Copy link
Contributor

@override
String version;

String installPath;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like this is never mutated - perhaps make it final?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. Done.

@devoncarew
Copy link
Contributor

lgtm

@danrubel danrubel merged commit 90f7afc into flutter:master Nov 14, 2016
@danrubel danrubel deleted the flutter-doctor-filter-intellij-install branch November 28, 2016 21:24
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

flutter doctor finds more installed IntelliJs than it should

2 participants