Skip to content
This repository has been archived by the owner on Aug 4, 2020. It is now read-only.

Add func-params-comma-dangle rule #64

Merged
merged 1 commit into from
Jun 21, 2016
Merged

Add func-params-comma-dangle rule #64

merged 1 commit into from
Jun 21, 2016

Conversation

zertosh
Copy link
Member

@zertosh zertosh commented Jun 20, 2016

Like comma-dangle, with the same options, but for trailing commas in function parameter lists.

There is a bug in the AST, where in arrows functions, the loc's for params with flow types only cover the argument identifier (or rest element), and not the flow type. So for this case, this rule is a noop. I'll have to investigate it separately.

@zertosh
Copy link
Member Author

zertosh commented Jun 20, 2016

^ @cpojer

@hzoo hzoo merged commit 54983e1 into babel:master Jun 21, 2016
ghost pushed a commit to facebookarchive/nuclide that referenced this pull request Jul 16, 2016
…or of eslint-plugin-babel's

Summary: We upstreamed a better version of this rule. babel/eslint-plugin-babel#64

Reviewed By: nmote

Differential Revision: D3573577

fbshipit-source-id: c7492d3f76d577f9bd77a57e3b90f3d9b7c9e8ab
@hzoo
Copy link
Member

hzoo commented Aug 17, 2016

babel/babylon#51 is fixed in the latest babylon

nicolo-ribaudo pushed a commit to babel/babel that referenced this pull request Nov 14, 2019
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.

2 participants