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
Fix: arrow-parens
supports type annotations (fixes #7406)
#7436
Conversation
@mysticatea, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nfroidure, @vitorbal and @not-an-aardvark to be potential reviewers. |
LGTM |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks, @mysticatea!
Just a heads up: we have moved to a new CLA checker on pull requests. Even if you've previously signed our CLA, we will need to you sign the new one. To do so, look at the status checks for licence/cla and click the "Details" link. Sorry for the inconvenience. |
Any idea when this'll make it into a release? :) |
@joemckie We usually do release every two weeks on Friday. So this PR will be released today. |
Awesome, thank you! |
BREAKING CHANGE: enabled rule arrow-parens
What is the purpose of this pull request? (put an "X" next to item)
[X] Bug fix (template)
See #7406 for the template.
What changes did you make? (Give an overview)
If there is a type annotation, we cannot remove parentheses around parameters of arrow functions. But
arrow-parens
has warned those.This PR fixes the bug.
I generated the ASTs with type annotations by http://astexplorer.net/
Is there anything you'd like reviewers to focus on?
Nothing in particular.