-
Notifications
You must be signed in to change notification settings - Fork 30
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
Consider omit_local_variable_types
for inclusion
#24
Comments
How about considering a variant of There is no need to work on perfecting the notion of 'trivial' here, we just need to recognize some specific cases, e.g.:
Surely we could add more elements to this list, but again: It is not required that we get a perfect list of trivial types, we just want a useful one. For a local variable with any other initializing expression a declared type would be tolerated (not required), we just want to get rid of the ones that are actually useless, and that is not true for some of the cases that are currently flagged by |
Currently, A scaled back version of that lint that only removes obvious types is an interesting idea, though... |
See also dart-lang/sdk#58375 for discussion along the lines of @eernstg's proposal above. |
omit_local_variable_types
for inclusion
The variant of |
The Dart style guide has long recommended
omit_local_variable_types
. This however is somewhat in conflict with Flutter samples and style.We should consider the path forward here, and if this can be included in recommended lints.
The text was updated successfully, but these errors were encountered: