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

06 - Advanced Functions #41

Closed
wants to merge 9 commits into from
Closed

06 - Advanced Functions #41

wants to merge 9 commits into from

Conversation

singularity-code
Copy link
Contributor

06 - function-object : 이관후 PR 취소되어 반영못한부분. 맞춤법체크
07 - new-function : 이관후 PR 취소되어 반영못한부분. 맞춤법체크
08 - settimeout-setinterval : 리뷰해주신후 수정한 내용
10 - bind (PR을 위해서 draft 했던 부분 취소)

문제있으면 알려주세요.
감사합니다.

@Violet-Bora-Lee
Copy link
Member

Violet-Bora-Lee commented May 17, 2019

크리스님 일단, 작업하시는 저장소가 항상 최신으로 유지되셔야해요.
현재 73커밋이 저장소에 반영되어있지 않습니다.
This branch is 9 commits ahead, 73 commits behind javascript-tutorial:master.

그리고 10 - bind (PR을 위해서 draft 했던 부분 취소) 에대한 커밋은 git revert, reset에 대해 자료를 찾아보시길 바랍니다.
http://www.devpools.kr/2017/02/05/%EC%B4%88%EB%B3%B4%EC%9A%A9-git-%EB%90%98%EB%8F%8C%EB%A6%AC%EA%B8%B0-reset-revert/

https://backlog.com/git-tutorial/kr/stepup/stepup7_2.html

@Violet-Bora-Lee
Copy link
Member

일전에 말씀드린 바와 같이 한 PR에 하나의 파일만 번역할 것을 권해드려요~
브랜치를 하나 만드셔서 해당 파일을 번역분이 담긴 commit을 cherry-pick하셔서 새로 만든 브랜치에 반영한 후, 그 브랜치를 PR하시면 될 것 같습니다.

@Violet-Bora-Lee
Copy link
Member

6,7,8번역분은 제가 직접 받아올게요..
앞으론 꼭 한 PR에 파일 하나만 해주세요!

changbokLee pushed a commit to changbokLee/ko.javascript.info that referenced this pull request Sep 24, 2020
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.

2 participants