adding unneeded semicolon & breaks standard

[stephenmathieson]

looks like an extra semi is being added when fat arrow function's params are wrapped with parens:

~/dev/src/github.com/segmentio/app on canary [!?] ∴ cat x.js
export default () => <div onSomething={(x) => y(x)}><foo /></div>

~/dev/src/github.com/segmentio/app on canary [!?] ∴ ./node_modules/.bin/standard-format x.js > y.js

~/dev/src/github.com/segmentio/app on canary [!?] ∴ ./node_modules/.bin/standard y.js
standard: Use JavaScript Standard Style (http://standardjs.com) 
  /Users/stephenmathieson/dev/src/github.com/segmentio/app/y.js:1:40: Parsing error: Unexpected token ;

~/dev/src/github.com/segmentio/app on canary [!?] ∴ cat y.js 
export default () => <div onSomething={;(x) => y(x)}>
                       <foo />
                     </div>

[carbureted]

+1, makes the formatter unusable for me :(

[carbureted]

It looks to me like the problem is probably best addressed in https://github.com/millermedeiros/esformatter-semicolon-first

[millermedeiros]

just released esformatter-semicolon-first@1.2.0 with the fix!

[feross]

Thanks!

[carbureted]

Feross, is the updated package working for you? I'm still getting the same bug when used in standard-format, but not when the esformatter-semicolon-first package is used independently. I may be making some dumb mistake, though.

[feross]

@carbureted Yeah, you're right -- it's not working for me either.

Sidenote: @maxogden -- can you please add @millermedeiros to this repo?

[feross]

Extra sidenote: @carbureted you might try using standard --fix (part of the latest standard beta (npm install standard@beta) instead of standard-format.

[carbureted]

@feross standard --fix is working for me, thanks!

[feross]

You should use standard --fix instead of this package. standard-format isn't being actively maintained anymore.