Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

arrow-parens: Allow binding patterns ([x, y]) => ... and ({x, y}) => ... to have parens #1958

Merged
merged 3 commits into from Jan 1, 2017

Conversation

andy-hanson
Copy link
Contributor

@andy-hanson andy-hanson commented Dec 31, 2016

PR checklist

What changes did you make?

Allowed binding patterns in arrow-parens.

Is there anything you'd like reviewers to focus on?

Why are async functions excluded from this rule?
EDIT: Looks like that was a kludge (#1479). Added support for async functions.

@@ -16,11 +18,14 @@ var piyo = <T, U>(method: () => T) => {
method();
};
const validAsync = async (param: any) => {};
const validAsync = async (param) => {};
const invalidAsync = async (param) => {};
Copy link
Contributor

Choose a reason for hiding this comment

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

move under comment // invalid case

@nchen63 nchen63 merged commit 7b6ac79 into palantir:master Jan 1, 2017
@nchen63
Copy link
Contributor

nchen63 commented Jan 1, 2017

@andy-hanson thanks!

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

Successfully merging this pull request may close these issues.

None yet

2 participants