-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Ko/equality comparisons and sameness #15011
Ko/equality comparisons and sameness #15011
Conversation
docs/ko/fully_untranslated_files/Web_JavaScript_Equality_comparisons_and_sameness.md
Outdated
Show resolved
Hide resolved
…risons_and_sameness.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Preview URLs Flaws (7)URL:
External URLs (3)URL:
(comment last updated: 2023-08-19 10:57:32) |
@hochan222 아무래도 한국어 리뷰가 다른 로케일에 비해 꾀나 늦는것 같습니다. 머지를 기다리는 PR이 제일 많기도 하고요.. 해결할 수 있는 방법이 없을까요..ㅜ |
@SnowMarble 님 안녕하세요. 먼저, PR이 방치되어서 정말 죄송합니다. 어떤 이유가 있더라도.. PR에 대한 리뷰가 늦어서 PR 병합이 늦어지는건 기여자의 입장으로 좋은 경험은 아니라는 사실 인지하고 있습니다. 그렇기에 다시 한번 죄송하다는 말씀을 드립니다. 보통 yari-content-ko팀에서는 2주까지 리뷰 드리는것을 목표로하고 있으며, 3주까지 마지노선으로 잡고 있습니다. 3주 이상된 PR에 대해서 리뷰가 지연되었다고 표현하고 있습니다. 모두 MDN을 전업으로 하고 있지 않기에 2주까지는 리뷰가 지연되어도 양해 부탁드립니다. 그렇지만 2주 이전 혹은 이후 언제든지 hochan222 멘션 주시면, 최대한 빠르게 리뷰 드릴 수 있도록 하겠습니다. 제가.. 보통 이런 상황에서 처음 기여하시는 분들의 리뷰를 먼저 드리다보니.. @SnowMarble님의 PR이 많이 방치되게 되었습니다. 죄송합니다. 현재 yari-content-ko 팀은 10명입니다. PR은 yari-content-ko 팀에 속한 인원에게 순차적으로 분배되고 있습니다. 1년 중 항상 모든 리뷰어들의 일정이 동일한것은 아니기에 특정 시기에 리뷰가 지연되는 상황이 발생 할 수 있습니다. 리뷰어 분들은 리뷰가 많이 쌓여도 최대한 동일한 품질의 리뷰를 유지하기 위해 노력하고 계십니다. 번역 PR의 경우, 보통 리뷰어분들이 리뷰를 진행할 때 content repo의 동일한 파일의 한문장, 한문장 확인하며 리뷰를 드립니다. 리뷰어분들중 리뷰를 드릴 시간이 부족하다면, 일정 조율을 통해 다른 분께 분배하거나 제가 대신 리뷰를 드리고 있습니다. 3주 이상 지난 PR에 대해서는 제가 적극적으로 확인하여 일정 조율을 하지 못한 책임이 큽니다. 최대한 한달 이상 지연되지 않도록, 조금 더 적극적으로 이야기 나누고 리뷰드리겠습니다.
멘션주신 뒤로 그동안의 리뷰들과 다른 로케일 현황을 보았습니다. 다른 로케일은 주로 PR의 대부분을 로케일팀이 생성하거나, 한국만큼 많은 PR이 들어오지 않습니다. 한국의 경우는 다른 로케일에 비해 새로운 기여자들이 기여를 해주시는 경우가 많습니다. 8월 11일 18시 기준 다른 로케일들에 비해 한국 로케일의 Open된 PR이 61개로 가장 많았는데요. 그중 3주가 넘은 PR은 11개였습니다. 지연되고 있는 PR 만큼 병합된 PR도 많다는 점.. 알아주셨으면 좋겠습니다.
많이 지연되는 리뷰 사항이 크지 않거나 PR중에 충돌만 해결되면 병합될 수 있는 PR이 있다면 리뷰어가 직접 충돌을 해결 후 병합하면 조금 더 지연되는 PR이 줄어들듯합니다. 조금 더 고민해보고 안내서에 추가하겠습니다. 50개의 PR에 대해서는.. 짧은 기간에 PR이 몰린 경우로 팀을 늘리는 해결책 외에는 떠오르지 않는데 혹시 의견 추후에라도 주시면 좋을듯합니다. 기여자 분들과 동일하게 리뷰어 분들도 바쁜 시간 중에 정성스럽게 리뷰를 드리려고 노력하고 계십니다. 많이 지연되는 PR이 있다면.. 제가 조율을 잘 못하고 있는 것이니 hochan222를 멘션주시면 최대한 빠르게 리뷰 드리도록 하겠습니다. 좋은 주말 보내세요 :) (cc. @mdn/yari-content-ko ) @IMHOJEONG 님, @rbals0445 님 같이 이모지로 의견 주셔서 감사합니다. |
@hochan222 최근에 활동을 못하고 있어 정말 조심스럽게 의견을 적어보았고, 저의 단순한 불평으로 오해하여 무례하게 보이지 않을까 걱정도 되었지만, 이렇게 긴 글로 정성스레 답변해 주셔서 정말 고맙습니다.! 🥹 리뷰어가 더 적극적으로 기여하는 경우와 달리 우리 로케일어서 새로운 기여자가 많은 것은 좋아해야 하는 상황이지만, 제가 단순히 지연된다 하여 리뷰어의 노고를 무시하는 것처럼 느끼지 않길 바라겠습니다ㅠ🙇♂️ 리뷰가 이루어지는 과정과 지연되는 이유를 포함하여 말씀해 주신 부분 모두 잘 이해하고 있고, 다른 기여자 분들도 그럴 거라 확신합니다. 그리고 한국어 로케 일을 꾸준히 모니터링하고 있어 최근에 더 힘쓰고 계신 것도 당연히 알고 있었습니다!🔥🔥 하지만, 여러 요인이 있음에도, 장기적으로 봤을 때 지연된 PR이 자주 존재했기 때문에, 지연되는 이유가 궁금했던 다른 분들에게도 분명 큰 참고가 되었을 것입니다. 갑자기 리뷰가 몰리는 상황은 물리적인 이유로 딱히 방법이 없을 것 같네요. 리뷰가 지연되는 경우엔 지연된다는 코멘트를 남겨 방치되고 있지 않다는 것만 보여주어도 완벽하다고 생각합니다. 저도 방치된 PR을 찾으면 적극적으로 맨션 드리도록 하겠습니다. 누구보다 적극적으로 활동해 주셔서 괜히 마음이 무거운데요, 다시 한번 감사드리고, 시원한 주말 보내세요 :) |
@SnowMarble 님, 항상 감사드립니다.
리뷰가 늦어지는 PR에 대해서는 문구를 정해 일정한 기간이 지나면 지정된 리뷰어가 해당 문구 커멘트로 남기는 방향으로 이야기 나누어 보겠습니다. @mdn/yari-content-ko 추가적인 의견 있으시면 말씀해주세요..! 저도 다시한번 감사드립니다..! 시원한 주말 보내세요 :) |
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
…x.md Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
files/ko/web/javascript/equality_comparisons_and_sameness/index.md
Outdated
Show resolved
Hide resolved
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
…x.md Co-authored-by: hochan Lee <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SnowMarble 님, 빠르게 반영해주셔서 감사합니다.
좋은 주말 보내세요 :)
@yujo11 #14076 깃 망해서 다시올립니다ㅎ
#14968