-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
What are the default lints enforced? #57872
Comments
Nice try, haha :) I'm doing my best to not think about style/linting 🙈 I mostly care that we have one set of rules we recommend (and maybe one for flutter), and that someone is willing to defend those rules. |
🥇
We're on the same page. That's exactly why I'm suggesting |
Should we be running any lints of none have been configured for a project? I think I would expect no lints to be enabled; just errors and warnings. We can guide people to best practices about which lints to have in the project creation stage - have stagehand create projects with the recommended best practices lints (ala pedantic). |
Me too. (I thought I'd responded to the thread saying as much, but I guess I forgot to click "Comment", so I'm glad you said it.) Lints are designed to be opt-in. I think it was a mistake to enable any lints by default, and I think we should fix that. |
Ah, okay that's a fine opinion... I just assumed that there was some sane default set of lints it would always run, and I wanted to know what these were :) |
Well, there is a default set of lints, but I don't know that I'd call it sane. :-) |
What are the default lints enforced...
Reading the
README
I would expect to learn what happens when I rundartanalyzer
. I don't configure lints for most of my projects, and just assume that some sane defaults are present.Reading the README:
Suggests that it might be that the defaults are:
But it's not clear to me that this is what it says. Could we possibly write a single line somewhere referencing the default lints?
The text was updated successfully, but these errors were encountered: