-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
New: arrow-body-style rule (fixes #4109) #4144
Conversation
LGTM, thank you @alberto ! |
Didn't you see #4109 (comment)? edit: Never mind, I see you opened this before that comment. |
@michaelficarra I'm not sure what you mean. I didn't include the other patterns I mentioned there. |
Sorry, I thought that comment was referring to the whole rule. |
LGTM. Going to hold off on merging in case we need another patch release. |
Great, thanks! |
Should this be |
Yeah, I like it better. @mysticatea? |
Yes, I like too. |
This looks good to go. We're going to do one more patch release, then I think we can merge this. |
@nzakas just a reminder that this one is ready |
New: arrow-body-style rule (fixes #4109)
No description provided.