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

Update README-ko.md #95

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README-ko.md #95

wants to merge 2 commits into from

Conversation

ohseunghyeon
Copy link

require.js로 불리는 프로젝트와 뭔가 할 것이 있다고 해석이 되어 있어서 관련이 있다로 해석을 변경하였습니다. 그리고 en 버전을 보니 그 윗문단이 추가되었기에 해석하여 추가하였습니다.

require.js로 불리는 프로젝트와 뭔가 할 것이 있다고 해석이 되어 있어서 관련이 있다로 해석을 변경하였습니다. 그리고 en 버전을 보니 그 윗문단이 추가되었기에 해석하여 추가하였습니다.
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 생태계에서도 일반적이지는 않습니다.

Choose a reason for hiding this comment

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

I think you've made a mistake here.

"생각하 혼란스러워합니다."

Copy link
Author

Choose a reason for hiding this comment

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

you are right.

Copy link
Author

Choose a reason for hiding this comment

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

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..

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.

None yet

2 participants