Update README-ko.md#95
Open
ohseunghyeon wants to merge 2 commits intopetehunt:masterfrom
ohseunghyeon:patch-2
Open
Update README-ko.md#95ohseunghyeon wants to merge 2 commits intopetehunt:masterfrom ohseunghyeon:patch-2
ohseunghyeon wants to merge 2 commits intopetehunt:masterfrom
ohseunghyeon:patch-2
Conversation
require.js로 불리는 프로젝트와 뭔가 할 것이 있다고 해석이 되어 있어서 관련이 있다로 해석을 변경하였습니다. 그리고 en 버전을 보니 그 윗문단이 추가되었기에 해석하여 추가하였습니다.
leehankyeol
suggested changes
Jan 11, 2018
README-ko.md
Outdated
| 명심할 점 : `CommonJS`는 `require()` 함수를 모듈 임포트에 사용합니다. 그래서 많은 사람들이 `require.js`로 불리는 프로젝트와 뭔가 해야하는 지 헷갈립니다. 여러가지 기술적 이유로 `require.js`를 쓰지 말것을 추천해 드립니다. React생태계에서도 일반적이지는 않습니다. | ||
| React는 현재 Create React App 이라는 공식 CLI tool을 제공합니다. 이 tool은 아무런 설정 없이 webpack을 사용하는 React 프로젝트를 만들게 해줍니다. 이 툴에도 한계는 있지만, 좋은 시작점을 제공하며 차후 업데이트를 통해 더욱 많은 기능을 갖게 될 것입니다. 또한 "ejection" 기능을 제공하는데, 기존의 설정과 의존성을 복사하여 당신의 프로젝트에 완전한 통제를 갖도록 합니다. | ||
|
|
||
| 명심할 점 : `CommonJS`는 `require()` 함수를 모듈 임포트에 사용합니다. 그래서 많은 사람들이 `require.js`로 불리는 프로젝트와 관련이 있다고 생각하 혼란스러워합니다. 여러가지 기술적 이유로 `require.js`를 쓰지 말것을 추천해 드립니다. React 생태계에서도 일반적이지는 않습니다. |
There was a problem hiding this comment.
I think you've made a mistake here.
"생각하 혼란스러워합니다."
Author
There was a problem hiding this comment.
i fixed it. and i worked on this website so another commit has been made. should i have done 'commit --amend'?
but actually i don't know how to do that on web..
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
require.js로 불리는 프로젝트와 뭔가 할 것이 있다고 해석이 되어 있어서 관련이 있다로 해석을 변경하였습니다. 그리고 en 버전을 보니 그 윗문단이 추가되었기에 해석하여 추가하였습니다.