-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: dev config tool update (fix #298) #295
Conversation
@jinwoo-kim-nhn 바꾸게된 이유와 변경 내역이 github issue로 PR에 연결되어 관리되면 좋을것 같습니다. |
fabric 설치 방법이 바뀐다면 readme에 설명도 바뀌어야 할 것 같습니다! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
테스트도 문제 없이 돌아가는거죠?ㅎㅎ
리뷰 완료입니다. 고생 많으셨어요~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[01/14] Reviewed. Keep up the good work!
karma.conf.js
Outdated
{ | ||
test: /\.js$/, | ||
include: /src/, | ||
exclude: /node_modules|bower_components/, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doest this use bower?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I knew wrong usage, bower did not need to excloud. Thank you.
karma.conf.js
Outdated
test: /\.styl$/, | ||
loader: 'css-loader!stylus-loader?paths=src/css/' | ||
use: [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A brief version is use: ['css-loader', 'stylus-loader']
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you.
webpack.config.js
Outdated
], | ||
loaders: [ | ||
{ | ||
exclude: /node_modules|bower_components/, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bower?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I knew wrong usage, bower did not need to excloud. OK
* feat: webpack version up * feat: cssExtract plugin update * feat: karma.conf update middle commit * feat: karma.conf update middle commit * feat: webpack with karma version update middle commit * update package-lock * feat: karma version update 4.x.x * fix: fixed all vulnerabilities at package.json * fix: added mode property webpack config * feat: added dist directory for npm istall test * test complete with delete dist directory * restore un nessary fix point * apply code review at webpack config * fix: fixed fabricjs install guide * fix: fix karma.conf.js for css-loader webpack config
Please check if the PR fulfills these requirements
fix #xxx[,#xxx]
, where "xxx" is the issue number)Description
issue
npm install fails to install normally on nodejs stable version #298
npm install fails to install normally on nodejs stable version
Thank you for your contribution to TOAST UI product. 🎉 😘 ✨