Skip to content

#45 function obj task#71

Merged
Violet-Bora-Lee merged 8 commits intojavascript-tutorial:masterfrom
singularity-code:#45_function_obj_task
Jun 29, 2019
Merged

#45 function obj task#71
Violet-Bora-Lee merged 8 commits intojavascript-tutorial:masterfrom
singularity-code:#45_function_obj_task

Conversation

@singularity-code
Copy link
Contributor

이관전 반영못한 부분인데 충돌되는 부분이 있는지 모르겠네요.
문제있으면 알려주세요.

Copy link
Member

@Violet-Bora-Lee Violet-Bora-Lee left a comment

Choose a reason for hiding this comment

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

리뷰 진행하였습니다 :)
항상 어려운 글 번역 맡아주심에 감사드립니다.

@singularity-code
Copy link
Contributor Author

리뷰 진행하였습니다 :)
항상 어려운 글 번역 맡아주심에 감사드립니다.

감사합니다. 간단한건 반영해서 Resolved 하고 검토가 필요한건 다시 확인해보고 알려드릴께요

@singularity-code
Copy link
Contributor Author

리뷰해주신 내용 반영했습니다. resolved로 닫지 않은건 적합한 표현인지 한번더 봐주세요.
감사합니다.

@Violet-Bora-Lee
Copy link
Member

conflict를 resolve하고 다시 PR주시기 바랍니다.

@singularity-code
Copy link
Contributor Author

Resolve 했습니다. 캬하하!


- `name` -- the function name. Usually taken from the function definition, but if there's none, JavaScript tries to guess it from the context (e.g. an assignment).
- `length` -- the number of arguments in the function definition. Rest parameters are not counted.
- `name` -- 함수의 이름. 함수를 정의할 때만 명시할 수 있는 게 아니라 할당하거나 객체 프로퍼티로도 가능합니다.
Copy link
Member

Choose a reason for hiding this comment

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

이 부분이 영어 원문에서 업데이트 되었는데, 번역이 누락되었네요.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

예리하십니닷!! 완전 놓치고 있었네요. 수정했습니다.

@Violet-Bora-Lee Violet-Bora-Lee merged commit 192bdf5 into javascript-tutorial:master Jun 29, 2019
@Violet-Bora-Lee
Copy link
Member

제가 리뷰가 많이 늦어져서 죄송합니다. 항상 어려운 부분 먼저 시작해주셔서 저도 열심히 공부하면서 따라가고 있네요 👍 멋지십니다 🥇

@singularity-code
Copy link
Contributor Author

아니에요. 괜찮습니다. 항상 엄격리뷰하셔서 놀라워용.
저야말로 아무것도 모르는데 많이 배우고 있어요. 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments