Skip to content
This repository has been archived by the owner on Feb 12, 2023. It is now read-only.

Reconsider lines_longer_than_80_chars #13

Closed
tenhobi opened this issue Jan 3, 2020 · 1 comment · Fixed by #21
Closed

Reconsider lines_longer_than_80_chars #13

tenhobi opened this issue Jan 3, 2020 · 1 comment · Fixed by #21
Labels
feedback wanted Looking for feedback from the community

Comments

@tenhobi
Copy link
Owner

tenhobi commented Jan 3, 2020

I would like to discuss and maybe reconsider adding the lines_longer_than_80_chars lint rule.

This package should aim to follow the Effective Dart guide as much as possible and disabling this lint rule is probably opinionated. Given the fact that you can easily suppress any lint rule per project, file or line, I would like to reconsider this and add it to the lints we use.

Cc @felangel as you have for now the biggest projects using this package.

@tenhobi tenhobi added the feedback wanted Looking for feedback from the community label Jan 3, 2020
@felangel
Copy link

felangel commented Jan 3, 2020

I agree that the rule should be added because as you mentioned the goal should be to match effective dart rules as closely as possible and people can always disable individual rules.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feedback wanted Looking for feedback from the community
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants