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

Implement --fix for avoid_single_cascade_in_expression_statements lint #757

Closed
cpelling opened this issue Nov 26, 2018 · 4 comments
Closed

Comments

@cpelling
Copy link
Contributor

It would be useful for dartfmt --fix to automatically fix violations of the lint avoid_single_cascade_in_expression_statements.

@matanlurey
Copy link
Contributor

It is extremely unlikely dartfmt would have knowledge of or how to fix lints (separate ecosystem).

@cpelling
Copy link
Contributor Author

For context, there's a discussion about enforcing this lint internally. Having an option similar to --fix-optional-new for automatically fixing cases which would violate that lint seems like it would be useful. I'm willing to help with the work.

@munificent
Copy link
Member

Yeah, this particular lint is one that's easy to recognize syntactically, which is why I told @cpelling that it was a good candidate for a --fix.

@munificent
Copy link
Member

Closing since we're getting rid of dart format --fix in favor of dart fix: #1153

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants