-
Notifications
You must be signed in to change notification settings - Fork 118
Pin pedantic lints to 1.8.0 #565
Comments
At a minimum, I think updating the templates to drop them is great. Not 100% sure about enabling the lints for end users, though... |
1.8.0 pedantic includes unnecessary_new & unnecessary_const, as does the stagehand options file. When we update stagehand (and its templates) to pedantic 1.8.0, we should remove duplicate declarations (such as unnecessary_new & unnecessary_const) from the the stagehand options file. |
We should avoid having a free-floating pedantic dependency, since that's likely to break code whenever devs do a If we continue to use pedantic, we should have something like this in each template's
Currently, the analysis options files have something like this (which I'm calling a free-floating pedantic dependency):
|
/cc @mit-mit |
I'd recommend pinning to pedantic 1.8.0 for now. We're investigating using a different default set of lints, so no reason to push additional lints right now. |
Related discussion: dart-lang/site-www#1016 (comment)
The text was updated successfully, but these errors were encountered: