[번역 및 충돌해결] Part1. 11.6-8 (C21)#1861
Conversation
|
@seoyoonyi 서윤님! 리뷰 부탁드립니다ㅎㅎ |
|
@hyemimi 넵넵! 오늘 알바도 있고 번역도 조금 남아 있어서, 빠르면 내일 새벽이나 저녁쯤 확인 가능할 것 같습니다. 지금 핸드폰으로 보고 있어서 설정한게 안보이는데, 그동안 서명이랑 리뷰어/담당자 설정 먼저 해주시면 될 것 같아요 :) ps. PC로 보니 할당안되고 리뷰어 지정도 없네요! 태그해주셔서 감사합니당 |
seoyoonyi
left a comment
There was a problem hiding this comment.
async의 06-promisify 파일 1개 리뷰 완료했습니다.
master 브랜치의 커밋 메시지를 보니 [내용] 메시지 형식으로 통일되어 있는 것 같습니다. 커밋 메시지도 해당 형식에 맞춰 수정 부탁드립니다.
seoyoonyi
left a comment
There was a problem hiding this comment.
async의
07-mircrotask-queue
08-async-await/02-rewrite-async-2/soultion
08-async-await/02-rewrite-async-2/task
08-async-await/03-async-from-regular/task
파일 4개 리뷰 완료했습니다.
seoyoonyi
left a comment
There was a problem hiding this comment.
async의
08-async-await/article
08-async-await/04-promise-all-failure/solution
08-async-await/04-promise-all-failure/task
파일 3개 리뷰 완료했습니다.
수정 후 태그 부탁드립니다.
hyemimi
left a comment
There was a problem hiding this comment.
@seoyoonyi 서윤님! 리뷰해주신 부분들 위주로 (번역 수정, 커밋 메시지 수정) 반영해서 올려두었습니다. 괜찮으시다면 수정된 부분들 확인해주시면 감사하겠습니다!
|
There was a problem hiding this comment.
@Violet-Bora-Lee 안녕하세요 보라님! 해당 파일(충돌은 나지 않았으나 새로 생성되어 한글 번역본 없음)의 경우 이번 충돌 해결에 같이 포함하면 되는지 여쭤보고 싶습니다. 다른 PR에서는 신규 번역의 경우 PR을 따로 분리하신 것 같아서 혼동이 되어 여쭤봅니다..!
There was a problem hiding this comment.
04-promise-all-failure/task.md 파일도 신규 번역입니다.
요약
Part1. 11.6-8 async 관련 문서의 영어 원문 동기화 충돌을 해결하고, Promise.all 관련 과제와 해설을 신규 번역했습니다.
수정 파일
1-js/11-async/06-promisify/article.md
1-js/11-async/07-microtask-queue/article.md
1-js/11-async/08-async-await/02-rewrite-async-2/solution.md
1-js/11-async/08-async-await/02-rewrite-async-2/task.md
1-js/11-async/08-async-await/03-async-from-regular/task.md
1-js/11-async/08-async-await/04-promise-all-failure/solution.md
1-js/11-async/08-async-await/04-promise-all-failure/task.md
1-js/11-async/08-async-await/article.md
Pull Request 체크리스트
TODO