Add lints for Flutter apps and packages #9
Conversation
goderbauer
force-pushed
the
addFlutterLints
branch
from
764af55 to
6f818dd
Compare
lib/flutter.yaml
Outdated
| @@ -0,0 +1,17 @@ | |||
| # Lints recommended for Flutter apps and packages. | |||
|
|
|||
| include: package:lints/recommend.yaml | |||
There was a problem hiding this comment.
"recommended.yaml" now.
Can this be a relative path, so just "recommended.yaml"?
There was a problem hiding this comment.
Changed to the new name.
It appears that a relative path works here, I was just following how "recommended.yaml" is importing core:
@pq Should we change that and this one to use the relative path?
There was a problem hiding this comment.
cc @bwilkerson who knows more about nuances of includes... Brian: any reason not to use a relative path import?
There was a problem hiding this comment.
I could see an argument for using the package:... reference style being that people cribbing off this file would be able to use if for their own analysis options files (a relative reference wouldn't work for them).
goderbauer
force-pushed
the
addFlutterLints
branch
from
6f818dd to
400c60b
Compare
|
Superseded by flutter/packages#343. |
Fixes dart-lang/sdk#58349.
/cc @pq @mit-mit @Hixie