Skip to content
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

Removing low-priority lint rules #5073

Open
srawlins opened this issue Aug 21, 2024 · 3 comments
Open

Removing low-priority lint rules #5073

srawlins opened this issue Aug 21, 2024 · 3 comments
Labels
P3 A lower priority bug or feature request set-recommended Affects a rule in the recommended Dart rule set type-task

Comments

@srawlins
Copy link
Member

srawlins commented Aug 21, 2024

If we land analyzer plugins well, we may choose to stop supporting our lowest-priority lint rules. Maybe lint rules that do not guard against errors or bloat, and are not in the lints:core or lints:recommended sets, and are not used by the Dart or Flutter teams or Google, and are not backed by Effective Dart.

If any team or user found any such removed rule useful, they could resurrect it themselves quite easily, as the new plugins API uses the existing API for defining a first-party lint rule.

I thought I'd jot down some ideas to see what the scope of this might be. I personally think we could remove:

@srawlins srawlins added type-task P3 A lower priority bug or feature request labels Aug 21, 2024
@github-actions github-actions bot added the set-recommended Affects a rule in the recommended Dart rule set label Aug 21, 2024
@lrhn
Copy link
Member

lrhn commented Sep 24, 2024

I'd say these ones do avoid bloat:

Not a lot of bloat, but still unnecessarily long code.

A little sad for eol_at_end_of_file, but that's something your editor should be doing anyway, so not really lint-worthy.
A few of the rest will be "good riddance" from me, but most really don't matter any more. If ever.
(SGTM!)

@eernstg
Copy link
Member

eernstg commented Sep 24, 2024

They look like local syntactic style related rules. Probably not something that anyone would miss a lot.

One exception: always_specify_types seems to be the target of a healthy dose of complaints, but is it really true that nobody cares about that one?

@pq
Copy link
Member

pq commented Sep 25, 2024

I love the idea of a clean up! If we do something like this, it'd be nice to get crisp on criteria for removal. Should it matter if the proposed rules are in broad adoption?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P3 A lower priority bug or feature request set-recommended Affects a rule in the recommended Dart rule set type-task
Projects
None yet
Development

No branches or pull requests

4 participants