Closed yunyoungJang closed 3 months ago
π 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!
thanks @yunyoungJang for initializing this file! @YewonCALLI can start collaborating on this as well.
μ€μλ @yunyoungJang λΉ λ₯Έμμ κ°μ¬λ립λλ€π©΅π μΈμ§λ μ λ μ€λ μ λ μκ° μ΄νλ‘ λ¦¬λ·° κ°λ₯ν©λλ€ @designerSejinOH
κ°μ¬ν©λλ€βΊοΈ μ€μλ @yunyoungJang!! μΈνλ μ λ λ°€(10μ μ΄ν)μ 리뷰 λ³Ό μμ μ λλ€~! @yinhwa
@yinhwa @designerSejinOH @ienground @wooknick μ κ° μ무λλ κ°λ°μ΄ μ£Ό μ κ³΅μ΄ μλμ¬μ κ·Έλ°μ§ μμ°¨λ²μ κ·Έλλ‘ μ¬μ©νλ λ¨μ΄λ, κ°λ° μμ λ΄ μ£Όλ‘ μ¬μ©νλ μ©μ΄κ° μ΄λ ΅λλΌκ΅¬μ. μ£Όλ‘ νΌμ κ°λ°κ³΅λΆλ₯Ό νκ±°λ μμλΌλ¦¬ νμ μ ν΄μμ λ²μμμ νΌμ©λκ² μμ ν μ μ΄ μμμ΅λλ€! μ°μ 'μ μ₯μ' -> '리ν¬μ§ν 리'λ‘ λ°κΏλλλ‘ ν κ²μ~~!
@designerSejinOH μΈμ§λ! μ€μλμ΄ 1μ°¨ 리뷰 λ°μ 컀λ°ν΄μ£Όμ μ, 2μ°¨ 리뷰 μ§ννμλ©΄ λ©λλ€!
@yinhwa λ΅! λ°λ‘ μ§ννκ² μ΅λλ€! λ Έν° κ°μ¬ν©λλ€!
@yunyoungJang μ€μλ! μλ κ°μ΄λλΌμΈμ λ°λΌ, νμ¬ μμ μ€μ΄μ yunyoungJang : patch-1 forkμ λν μ°κΈ° κΆνμ @yewonCALLI μμλκ» κ³΅μ λΆνλ립λλ€! μ°κΈ° κΆνμ΄ κ³΅μ λ μ΄νμ μμλμ΄ μμ μμ κ°λ₯ν©λλ€ :)
Maintainer Kenneth Lim λ 곡μ κ°μ΄λλΌμΈ (24.03.18) : https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/managing-repository-settings/managing-teams-and-people-with-access-to-your-repository
@YewonCALLI μμλ κΆνμμ²κ°λμ§ νμΈλΆνλλ €μ!
@yinhwa, @designerSejinOH κΆν μμ² νμΈλΆνλ립λλ€ π
@yinhwa, @designerSejinOH κΆν μμ² νμΈλΆνλ립λλ€ π
κΆν μΉμΈνμ΅λλ€. κ°μ¬ν©λλ€!
μ νν conversation resolve λ²νΌμ΄ 보μ΄μ§ μλ κ² κ°μμγ γ !! @designerSejinOH λμ΄ μμ ν΄μ£Όμ λΆλΆ λ°μνμ¬ μ»€λ°μμ΅λλ€.
Looks great, thanks @yunyoungJang @designerSejinOH @yinhwa @YewonCALLI π @all-contributors please add @yunyoungJang for translation
@Qianqianye
I've put up a pull request to add @yunyoungJang! :tada:
@all-contributors please add @YewonCALLI for translation
@Qianqianye
I've put up a pull request to add @YewonCALLI! :tada:
@all-contributors please add @designerSejinOH for translation, review
@Qianqianye
I've put up a pull request to add @designerSejinOH! :tada:
Hi @yunyoungJang @yinhwa @designerSejinOH, we just noticed that all the images are not showing on the ko/contributor_guidelines.md or the other translated Korean contributor docs from this week.
The English version of the contributor docs and the images folder are in the contributor_docs parent folder, and the Korean translated docs are in the contributor_docs/ko child folder. So we need to reconfigure the image paths. Sorry we didn't catch this issue earlier.
Could you ask the contributors to add '.../' at front of the image the path, so it'd be '../images/xxxx.png'? You can see es/contributor_guidelines.md as example. Thank you!
Hi @yunyoungJang @yinhwa @designerSejinOH, we just noticed that all the images are not showing on the ko/contributor_guidelines.md or the other translated Korean contributor docs from this week.
The English version of the contributor docs and the images folder are in the contributor_docs parent folder, and the Korean translated docs are in the contributor_docs/ko child folder. So we need to reconfigure the image paths. Sorry we didn't catch this issue earlier.
Could you ask the contributors to add '.../' at front of the image the path, so it'd be '../images/xxxx.png'? You can see es/contributor_guidelines.md as example. Thank you!
Thanks Q for figuring this out, and sorry we missed this out. We should've double checked on this. Could any of @yunyoungJang or @YewonCALLI kindly edit the image paths and pull a new PR on this? Thank you all π
@designerSejinOH
@yinhwa νΉμλ ν΄μ νμΈν΄λ³΄λ μ κ° λ΄λΉνλ λ¬Έμλ μ΄λ―Έμ§κ° κΉ¨μ Έμλ€μ.. λ€μ PR μ¬λ¦¬κ² μ΅λλ€
@wooknick νμΈν΄λ΄μ£Όμ μ κ°μ¬ν©λλ€..! μμ PRμ¬λ¦¬μλ©΄ λ°λ‘ νμΈν΄μ λ¨Έμ§ μμ²λ릴κ²μ!
@ienground @Orwiss νμ°λ, μ±νλλ μ£μ‘νμ§λ§, μμ ν΄μ£Όμ ¨λ λ¬Έμμ μ΄λ―Έμ§ κ²½λ‘ ν λ² νμΈ λΆνλλ €λ λ κΉμ?? μ΄λ―Έμ§ κ²½λ‘λ '../images/xxxx.png' μ΄λ° μμΌλ‘ μ ν¬ ko ν΄λ μμμ μλ κ²½λ‘λ‘ μμ ν΄μ£Όμλ©΄ λ κ² κ°μ΅λλ€..! μμ ν PR μ¬λ €μ£Όμλ©΄ λ°λ‘ νμΈνκ² μ΅λλ€!! κ°μ¬ν©λλ€π
@designerSejinOH @ienground μ κ° μ§κΈ μμ ν κ²μ!
λ¬Έμμ μ λ μμλ λΆλΆ λͺ¨λ μμ κ²½λ‘λ‘ μμ νμ΅λλ€. @YewonCALLI μμλλ νμΈλΆνλλ €μ! @designerSejinOH @yinhwa
οΏ½λ¬Έμμ μ λ μμλ λΆλΆ λͺ¨λ μμ κ²½λ‘λ‘ μμ νμ΅λλ€. @YewonCALLI μμλλ νμΈλΆνλλ €μ! @designerSejinOH @yinhwa
@yunyoungJang λ΅λ΅! μ λ νμΈν΄λ³΄κ² μ΅λλ€! νΉμ μ΄λμμ μμ ν΄μ£Όμ ¨λμ??
@designerSejinOH
https://github.com/processing/p5.js/compare/main...yunyoungJang:p5.js:patch-2
μμ λ§ν¬λ‘ λ€μ΄κ°μλ©΄ λ©λλ€. μ κ° κΉνλΈ νμ
μ΄ μμ§ λ―μ€μ΄μ μ΄λ €μμ΄ λ§λ€μ. μμ ν΄μΌν λΆλΆμ΄ μλ€λ©΄ μλ €μ£ΌμΈμ.
@yunyoungJang μ μλμμ!! μ ν΄μ£Όμ ¨μ΄μ. κ°μ¬ν©λλ€βΊοΈ "Create pull request"λλ¬μ μ λ²μ²λΌ PR μ¬λ €μ£Όμλ©΄ κ±°κΈ°μ νμΈνκ³ μΉμΈλ릴κ²μ!
Resolves #[Add issue number here]
Changes:
Screenshots of the change:
PR Checklist
npm run lint
passes