-
Notifications
You must be signed in to change notification settings - Fork 0
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: Add React&Redux boilerplate #1
base: for-review
Are you sure you want to change the base?
Conversation
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/02] 수고하셨습니다. 설정 파일 및 README 관련은 공통적인 것이 많아서 따로 작성하지 않고 es6 보일러플레이트의 리뷰를 참고하시면 되겠습니다.
jinwoo-kim-nhn/es6-boilerplate#1
- .eslintrc
- .prettierrc
- webpack.config.js
- package.json
- README.md
- example/index.html
"@babel/env", | ||
{ | ||
"modules": "false", | ||
"targets": { |
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.
진우전임 작업한 쪽하고 스타일이 다른데 논의해서 통일하시죠.
{
"presets": [
["@babel/env", {
"modules": "umd",
"targets": {
"browsers": ["last 2 versions", "not ie <= 8"]
}
}]
],
"plugins": ["@babel/plugin-proposal-object-rest-spread"]
}
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.
넵
@@ -0,0 +1,37 @@ | |||
module.exports = { |
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.
다른 보일러플레이트와 의논해서 스타일 맞추어요.
https://github.com/jinwoo-kim-nhn/es6-boilerplate/blob/540c3924a379444596f91483bd09d9d31cc712f6/.eslintrc.js
node: true, | ||
es6: true | ||
}, | ||
extends: ['tui', 'plugin:react/recommended', 'plugin:prettier/recommended'], |
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.
넵
extends: ['tui', 'plugin:react/recommended', 'plugin:prettier/recommended'], | ||
plugins: ['react', 'prettier'], | ||
rules: { | ||
indent: [ |
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.
네. 여기도 react plugin 제외하고 ES6와 동일한데요, 논의후 수정하겠습니다.
@@ -0,0 +1,8 @@ | |||
{ | |||
"trailingComma": "none", |
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.
별도의 주석 가이드가 필요한가요?
es6보일러플레이트에서 그대로 가져왔는데 이부분은 논의해서 제거하거나 필요한부분만 남기겠습니다.
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.
es6쪽에 변경사항 그대로 반영하면 될거 같아요.
"build": "webpack --mode production", | ||
"dev": "webpack-dev-server --open --mode development" | ||
}, | ||
"author": "Junghwan-Park", |
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.
리포옮기면 dl 로 바꿔 주세요~
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.
미리 바꿔놓을게요 ㅎㅎ
"scripts": { | ||
"build-dev": "webpack --mode development", | ||
"build": "webpack --mode production", | ||
"dev": "webpack-dev-server --open --mode development" |
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.
lint 스크립트가 빠졌네요
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.
넵
"webpack-dev-server": "^3.1.10" | ||
}, | ||
"dependencies": { | ||
"@babel/polyfill": "^7.0.0", |
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.
번들될 때 쓰나요?
@@ -0,0 +1,21 @@ | |||
/** |
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.
넵
/> | ||
<br /> | ||
<button type="button" className="button" onClick={this.onClickApplyButton.bind(this)}> | ||
적용 |
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.
네넵 ㅎㅎ
export default combineReducers({ | ||
greetingText, | ||
userName | ||
}); |
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.
보일러플레이트를 어디까지 제공해야 할 지가 고민이네요.
combineReducers
예제까지 제공하는 게 약간 과할 것 같긴 합니다.
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.
그럼 그냥 단일 리듀서로 갈까요?
<html> | ||
<head> | ||
<meta charset="UTF-8"> | ||
<title>Development</title> |
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.
넵.
다른 타이틀로 수정하라는 말씀이죠?
고생많으셨어요. js 소스에는 별 문제가 없구요, |
No description provided.