use the correct dependency for babel-plugin-dynamic-import-node

[vikr01]

babel-plugin-transform-dynamic-import was just a copy of babel-plugin-dynamic-import-node, and it's not been updated for a while while babel-plugin-dynamic-import-node has.

[facebook-github-bot]

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed.

If you have received this in error or have any questions, please contact us at [email protected]. Thanks!

[facebook-github-bot]

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

[Timer]

This was intentional because the package you switched to does not support Babel 7. When it supports Babel 7, we're more than happy to switch back to it!

[vikr01]

Makes sense.

PR'd airbnb/babel-plugin-dynamic-import-node#64

[bjankord]

Would love to see this in the v2.0.0 stable release if possible. Currently, @vikr01 is waiting on some PR's downstream to be reviewed/merged by the airbnb team.

[Timer]

This isn't going to stop us from releasing 2.0.0 as stable, as we already updated our fork of this plugin to be babel 7 compatible.
This change can be released in a patch version from our end, though.

[wolfy1339]

Now that v2.2 of babel-plugin-dynamic-import-node added support for Babel v7 (while still maintaining v6 compatibility), this can be updated and merged

[Timer]

Excellent, thanks for letting us know!

[vikr01]

Updated the version for babel-plugin-dynamic-import-node and changed target branch/rebased against master, since that seems to be what's current.