React-native-ui-kitten: RN 0.56 - bundle error "A trailing comma is not permitted after the rest element"

[michaltaberski]

Issue type

• [X] bug report
• [ ] feature request

Issue description

Right after init plain, fresh react-native app (RN version 0.56) and adding react-native-ui-kitten I am getting following error:

/node_modules/react-native-ui-kitten/src/components/image/rkModalImg.js: A trailing comma is not permitted after the rest element (277:17)

  275 |       index,
  276 |       style: imgStyle,
> 277 |       ...imgProps,
      |                  ^
  278 |     } = this.props;

[artyorsh]

Hi @michaltaberski,

Thanks for your notice :)
We have already done a great work on updating react and expo to latest versions and fixing lint issues.
These changes are currently available in master branch and will be released soon.

[wazho]

Dear @artyorsh ,

Is this issue fixed? I encounter the same problem and it cause compilation fail.

Diff

I have removed the comma in file src/components/image/rkModalImag.js can work successfully.

276 |         style: imgStyle,
277 | -       ...imgProps,
277 | +       ...imgProps
278 |     } = this.props;

Version of Kitten

$ yarn info v1.5.1
{ "name": "react-native-ui-kitten",
  "description": "React Native components",
  "dist-tags":
   { "latest": "3.0.1" },
   ...
}

Thanks.

[wazho]

I had seen the others related issues.

Hope the new version will release soon.

Thanks :)

[ranjeethpt]

Same here +1

[GiarcEnoredlac]

+1 as well

[prasadpilla]

I am waiting for the new version as well!

[usfslk]

You have to delete that comma next to imgProps but everytime you rebuild it's gonna happen again.

[prasadpilla]

@usfslk yes, that's why I am waiting for the fix. Delayed the upgrade for now

[usfslk]

@usfslk yes, that's why I am waiting for the fix. Delayed the upgrade for now

Why does it take so much time to remove a comma and push a commit? I'm new to these report issues stuff, there's also many libraries that break my code just by installing and importing them