-
Notifications
You must be signed in to change notification settings - Fork 7
Sync/cra 4.0 #24
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sync/cra 4.0 #24
Conversation
Co-authored-by: Ian Schmitz <ianschmitz@gmail.com>
Bumps [acorn](https://github.com/acornjs/acorn) from 6.4.0 to 6.4.1. - [Release notes](https://github.com/acornjs/acorn/releases) - [Commits](acornjs/acorn@6.4.0...6.4.1) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Passing an array with a single entry is not equivalent. This causes Webpack to generate another wrapper module around the entry. This is just unnecessary overhead and bytes.
See facebook#7790 for a previous similar PR.
This reverts commit c03bb36.
Co-authored-by: Ian Schmitz <ianschmitz@gmail.com>
Co-authored-by: Ian Schmitz <ianschmitz@gmail.com>
bae5ce7
to
0446591
Compare
Found a bug... For some reason in my itwin-viewer test app, im not seeing __IMODELJS_INTERNALS_DO_NOT_USE, and looking at the webpack.config I do see the IModeljsLibraryExportsPlugin being used. Need to investigate further |
NVM, disregard this. My sample app had both our bentley fork and the fb react-scripts as dev deps, and when i was running the app it defaulted to using the fb one, and thats why the imjs globals were missing. |
…t-refresh-webpack-plugin (facebook#9872)
Some library --> Some libraries
Co-authored-by: mad-jose <joset@yeswearemad.com>
Resolves facebook#10084 security vulnerability in websocket-driver library version 0.5.6, imported transitively by sockjs
Resolves facebook#10411 Bumps immer version to 8.0.1 to address the prototype pollution vulnerability with the current 7.0.9 version.
close in favor of #35 |
Syncing fork with upstream 4.0 release
only conflicts were with the following files: