Skip to content
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

fix: bootstrap failures #2187

Merged
merged 2 commits into from
Apr 14, 2021
Merged

fix: bootstrap failures #2187

merged 2 commits into from
Apr 14, 2021

Conversation

willworks
Copy link

@willworks willworks commented Apr 7, 2021

  • 修复本地 npm start 启动内存不足
  • 修复本地启动 isObject is not a function 问题

@willworks willworks changed the title fix: increase memory limit to 3gb for webpack devServer fix: bootstrap failures Apr 7, 2021
@@ -36,7 +36,11 @@ module.exports = options => ({
},
{
test: /\.js$/, // Transform all .js files required somewhere with Babel
use: 'happypack/loader?id=js'
exclude: [
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是否可以直接使用: exclude: /node_modules/

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个我也考虑过,davinci 依赖比较多,直接这样改的话,假设有些库没有按照标准实现或者直接使用 ES6/7 的特性,那就会有很多问题了

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

本人前端小白,最近也在看这个问题,之前定位到了core-js,目前只把core-js exclude是也是可以的~ 这样做是否可行呢?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

可行的,corejs 是标准库,不需要二次 babel

@scottsut scottsut merged commit f2ffc40 into edp963:master Apr 14, 2021
@scottsut
Copy link
Member

Thanks for the contribution!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants