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

add avoid_multiple_declarations_per_line #2502

Conversation

lukepighetti
Copy link
Contributor

Description

Lint on multiple variable declarations in a single line.

Fixes # (issue)

#2501

@coveralls
Copy link

coveralls commented Mar 11, 2021

Coverage Status

Coverage decreased (-0.04%) to 96.683% when pulling eec724b on lukepighetti:lukepighetti/avoid_multiple_declarations_per_line into e929624 on dart-lang:master.

@bwilkerson
Copy link
Member

@pq I don't know what the issue with pana is. Could you take a look at it?

@pq
Copy link
Member

pq commented Mar 11, 2021

Yeah, I need to look at those (new) build crashers. Tracking: #2504

@pq
Copy link
Member

pq commented Mar 11, 2021

@lukepighetti: what's a preferred email? I'd like to add an attribution to AUTHORS.

Thanks for the contribution!

Copy link
Member

@pq pq left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💎

@pq pq merged commit dff708b into dart-lang:master Mar 11, 2021
@lukepighetti
Copy link
Contributor Author

[email protected] thanks!

mockturtl added a commit to mockturtl/tidy that referenced this pull request Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants