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

[p5.js KO] add korean translation of 'contributing_to_the_p5.js_reference.md #6868

Merged
merged 6 commits into from
Mar 19, 2024

Conversation

wooknick
Copy link
Contributor

@wooknick wooknick commented Mar 15, 2024

Changes:
contributing_to_the_p5.js_reference.md 파일 추가 및 번역

PR Checklist

  • npm run lint passes
  • [Inline documentation] is included / updated
  • [Unit tests] are included / updated

Copy link

welcome bot commented Mar 15, 2024

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already. And be sure to add yourself to the list of contributors on the readme page!

@wooknick
Copy link
Contributor Author

@yinhwa @designerSejinOH

리뷰 부탁드립니다:)

@yinhwa
Copy link
Contributor

yinhwa commented Mar 16, 2024

@wooknick 민욱님 초스피드 번역 넘 멋지고 감사드립니다 😎✨✨ 리뷰는 곧 진행하겠습니다!

@designerSejinOH
Copy link
Contributor

@wooknick 빠른 과제 수행 감사드립니다..!! 저도 곧 리뷰하겠습니다🥳

Copy link
Contributor

@yinhwa yinhwa left a comment

Choose a reason for hiding this comment

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

@wooknick 민욱님의 문장력에 감탄하며 1차 리뷰 완료하였습니다! :) 빠른 리뷰 감사드려요! 이어서 @designerSejinOH 세진님이 2차 리뷰 진행할 예정입니다. 엇 근데 리뷰 코멘트 하나가 사라진것 같은데, 모든 "에셋"은 "애셋"으로 변경해주시면 됩니다! (참고: https://www.korean.go.kr/front/onlineQna/onlineQnaView.do?mn_id=216&qna_seq=266568&pageIndex=1)

@wooknick wooknick changed the title "add" korean translation of 'contributing_to_the_p5.js_reference.md [p5.js KO] add korean translation of 'contributing_to_the_p5.js_reference.md Mar 16, 2024
@wooknick wooknick requested a review from yinhwa March 16, 2024 09:40
@wooknick
Copy link
Contributor Author

@yinhwa 님 감사합니다:) 리뷰해주신 내용 모두 확인했습니다. 리뷰 내용 중에 한 가지만 빼고 모두 반영하여 resolve 해두었고, 아직 반영하지 않은 부분은 해당 리뷰에 댓글로 의견을 남겨뒀습니다. 확인 부탁드립니다!

@yinhwa
Copy link
Contributor

yinhwa commented Mar 18, 2024

@designerSejinOH 세진님! 민욱님도 1차 리뷰 반영 커밋 완료하셔서 2차 리뷰 진행하시면 됩니다 :) (중간에 maintainer님들께 질문남긴 코멘트만 제하고 봐주시면 될 것 같아요!)

@designerSejinOH
Copy link
Contributor

@wooknick 수정사항 반영 감사드립니다..!
@yinhwa 넵! 윤영님 2차리뷰 후 바로 민욱님 2차 리뷰로 넘어오겠습니다 :)

Copy link
Contributor

@designerSejinOH designerSejinOH left a comment

Choose a reason for hiding this comment

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

@wooknick 2차 리뷰 완료하였습니다.! 문의하셨던 부분에 대해서만 코멘트 남겨놓았고 다른 부분들은 다 좋은 것 같습니다..! 민욱님께서 계속 읽어보시면서 이해하기 쉬운 문장으로 잘 다듬어주신 것 같아 처음 읽는 저도 편하게 읽을 수 있어서 좋았습니다 :)
감사합니다! 고생하셨어요!
코멘트 부분만 수정해서 올려주시면 Approve 진행하겠습니다 ☺️
(2차 리뷰 소요 시간: 2024.03.18 23:30-24:20, 50분)

원문의 순서와 일치하도록 번역문 순서 변경
@wooknick
Copy link
Contributor Author

@designerSejinOH
리뷰 감사합니다! 의견 주신대로 원문의 순서와 일치하도록 수정했습니다.

@yinhwa
1차 리뷰해주신 내용 반영하고 나서 제가 인화님에게 리뷰를 re-request 한 것 같은데, 그래서인지 인화님 리뷰가 다시 awaiting으로 뜹니다.😅 이 부분 한 번만 확인 부탁드리겠습니다.

@Qianqianye Qianqianye added the Translation Translation of p5.js documentation label Mar 18, 2024
Copy link
Contributor

@designerSejinOH designerSejinOH left a comment

Choose a reason for hiding this comment

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

Hi @Qianqianye @limzykenneth, we have approved this PR!
We're ready to merge ☺️

@Qianqianye Qianqianye merged commit f5d8360 into processing:main Mar 19, 2024
2 checks passed
@Qianqianye
Copy link
Contributor

Looks great, thanks @wooknick @designerSejinOH @yinhwa! 💗
@all-contributors please add @wooknick for translation

Copy link
Contributor

@Qianqianye

I've put up a pull request to add @wooknick! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Translation Translation of p5.js documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants