Skip to content

[문서] 6.7~7.2 충돌 해결 및 미번역분 번역(C18)#1870

Open
meanzzi wants to merge 13 commits into
javascript-tutorial:2026-en-mergefrom
meanzzi:fix/C18-meanzzi
Open

[문서] 6.7~7.2 충돌 해결 및 미번역분 번역(C18)#1870
meanzzi wants to merge 13 commits into
javascript-tutorial:2026-en-mergefrom
meanzzi:fix/C18-meanzzi

Conversation

@meanzzi
Copy link
Copy Markdown

@meanzzi meanzzi commented May 17, 2026

요약

C18 담당 파일 충돌 해결 및 번역 수정

  • 변경 파일

    • 1-js/06-advanced-functions/07-new-function/article.md
    • 1-js/06-advanced-functions/08-settimeout-setinterval/article.md
    • 1-js/06-advanced-functions/09-call-apply-decorators/article.md
    • 1-js/06-advanced-functions/10-bind/5-question-use-bind/solution.md
    • 1-js/06-advanced-functions/10-bind/article.md
    • 1-js/06-advanced-functions/12-arrow-functions/article.md
    • 1-js/07-object-properties/01-property-descriptors/article.md
    • 1-js/07-object-properties/02-property-accessors/article.md

  • 작업 내용

    • 충돌 마커 제거
    • upstream 영어 원문 변경사항 한국어에 반영

연관 이슈

(fix #일이삼)

Pull Request 체크리스트

TODO

  • 번역 규칙을 확인하셨나요?
    • 줄 바꿈과 단락을 '원문과 동일하게' 유지하셨나요?
    • 맞춤법 검사기로 맞춤법을 확인하셨나요?
    • 마크다운 문법에 사용되는 공백(스페이스), 큰따옴표("), 작은따옴표('), 대시(-), 백틱(`) 등의 특수문자는 그대로 두셨나요?
  • 로컬 서버 세팅 후 최종 결과물을 확인해 보셨나요?
  • PR 하나엔 번역문 하나만 넣으셨나요?
  • 의미 있는 커밋 메시지를 작성하셨나요?
    • 예시
      • [프락시] 번역
      • [프락시] 과제 번역
      • [if문과 조건부 연산자 '?'] 리뷰
      • [주석] 2차 리뷰
      • [Date 객체와 날짜] 번역

@javascript-translate-bot javascript-translate-bot requested a review from a team May 17, 2026 01:47
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 17, 2026 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@meanzzi
Copy link
Copy Markdown
Author

meanzzi commented May 17, 2026

안녕하세요! @wonza-hub 님 리뷰 부탁드립니다!

wonza-hub
wonza-hub reviewed May 20, 2026
View reviewed changes
Copy link
Copy Markdown

@wonza-hub wonza-hub left a comment •
edited
Loading

Choose a reason for hiding this comment

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

확인했습니다:)
커밋 별로 리뷰 및 의견 달아드렸습니다.
참고부탁드립니다.

wonza-hub
wonza-hub reviewed May 21, 2026
View reviewed changes
Copy link
Copy Markdown

@wonza-hub wonza-hub left a comment

Choose a reason for hiding this comment

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

리뷰 확인부탁드려요:)

Comment thread 1-js/06-advanced-functions/08-settimeout-setinterval/article.md Outdated
Comment thread 1-js/06-advanced-functions/08-settimeout-setinterval/article.md Outdated
Comment thread 1-js/06-advanced-functions/09-call-apply-decorators/article.md Outdated
Comment thread 1-js/06-advanced-functions/10-bind/article.md Outdated
Comment thread 1-js/06-advanced-functions/10-bind/article.md
say("Bye"); // Bye, John! ("Bye" is passed to say)
>>>>>>> upstream/master
say("Hello"); // Hello, John! (인수 "Hello"를 say에 전달합니다.)
say("Bye"); // Bye, John! ("Bye"를 say에 전달합니다.)
Copy link
Copy Markdown

@wonza-hub wonza-hub May 20, 2026

Choose a reason for hiding this comment

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

기존 수동문을 능동문으로 변경하신 이유가 궁금합니다.

둘의 차이가 크지 않지만, 윗부분에도 대체로 수동문으로 되어있기 때문에 통일시키는 것이 좋아보입니다.

Copy link
Copy Markdown
Author

@meanzzi meanzzi May 21, 2026

Choose a reason for hiding this comment

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

피동형 문장에 주의하세요.
사실 원칙대로라면 ~되다같은 수동태도 가능하면 억제하는 편이 좋지만 실제로 작업을 하다 보면 써야 문장이 자연스러울 때도 있습니다. 다만 보여지다, 생각되다와 같은 불필요한 피동형 문장은 자제하는 편이 좋습니다.

위의 번역 모범 사례를 참고하여 능동문으로 변경하였습니다. 다만 말씀해주신 것처럼 윗부분이 전반적으로 수동문 형태로 작성되어 있어 표현을 통일하는 방향도 고민되는 부분이라, 어느 쪽이 더 적절할지 의견 주시면 그에 맞춰 반영하겠습니다!

Copy link
Copy Markdown

@wonza-hub wonza-hub May 21, 2026

Choose a reason for hiding this comment

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

말씀주신대로 번역 모범 사례를 따르면 좋겠지만, 그럴 경우 문서 전반적으로 수정범위가 넓어질 것 같습니다.
이 부분 또한 메인테이너 분께 한 번 더 검토 부탁드리는 게 좋을 것 같습니다:)

Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
Comment thread 1-js/07-object-properties/01-property-descriptors/article.md Outdated
wonza-hub
wonza-hub reviewed May 21, 2026
View reviewed changes
Copy link
Copy Markdown

@wonza-hub wonza-hub left a comment

Choose a reason for hiding this comment

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

Part1 7.2 프로퍼티 getter와 setter 부분 확인했습니다.

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

Reviewers

1 more reviewer

@wonza-hub wonza-hub wonza-hub left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

review needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@meanzzi @CLAassistant @wonza-hub @javascript-translate-bot