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

[ko] Translate 'Cluster' #1131

Merged
merged 5 commits into from
Sep 6, 2022
Merged

[ko] Translate 'Cluster' #1131

merged 5 commits into from
Sep 6, 2022

Conversation

kidae92
Copy link
Contributor

@kidae92 kidae92 commented Jul 28, 2022

Relevant with #1116 (comment)

@netlify
Copy link

netlify bot commented Jul 28, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit 679ff52
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/631552398b27440008d141e2
😎 Deploy Preview https://deploy-preview-1131--cncfglossary.netlify.app/ko/cluster
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@yunkon-kim yunkon-kim added the lang/ko for Korean label Jul 28, 2022
@yunkon-kim
Copy link
Collaborator

@kidae92 기여 감사합니다 ^^

DCO와 관련하여 해결해주실 부분이 있습니다. 현재 열려있는 PR을 참고하시기 바랍니다.

Signed-off-by: kidae92 <kidae92@hanmail.net>
Copy link
Collaborator

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

@kidae92 리뷰 의견을 드립니다 ^^

전반적으로 매끄럽게 다듬는 차원의 의견을 드렸습니다.

기여자께서 판단하시어 적용할 것은 적용하시고, 논의할 내용은 답글로 남겨주시면 되겠습니다.

content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
@yunkon-kim
Copy link
Collaborator

PTAL @kidae92 안녕하세요 ^^

첫 번쨰로 드린 리뷰 코멘트를 반영하신 후에, 두 번째 리뷰가 진행될 예정입니다. 참고하시기 바라며, 혹시 리뷰 코멘트 해결에 이슈가 있으시면 말씀해주십시오.

Signed-off-by: kidae92 <kidae92@hanmail.net>
Signed-off-by: kidae92 <kidae92@hanmail.net>
Copy link
Collaborator

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

LGTM !!

@kidae92 리뷰 의견 반영하시느라 고생하셨습니다 👍

Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

@kidae92 기여 감사합니다! :)

리뷰 의견 참고하셔서 PR 업데이트를 부탁드립니다.

content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
content/ko/cluster.md Outdated Show resolved Hide resolved
Signed-off-by: kidae92 <kidae92@hanmail.net>
content/ko/cluster.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

LGTM :)

@kidae92
마지막에 추가된 커밋에 sign-off 가 빠져 있어서,
DCO가 실패되고 있습니다. :)

가능하시면 커밋을 1개로 스쿼싱해주시면서, sign-off가 되도록 처리 부탁 드립니다.
(또는 @yunkon-kim 께서 해당 PR을 스쿼시 머지 방식으로 DCO에 문제없도록 처리해주셔도 괜찮을 것 같습니다. 향후 가이드는 @yunkon-kim 님께 맡기겠습니다. ㅎㅎ)

@yunkon-kim
Copy link
Collaborator

@seokho-son @kidae92
Squash and merge 하면서 DCO 처리하겠습니다 ^^

@yunkon-kim yunkon-kim merged commit 1e12c22 into cncf:dev-ko Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/ko for Korean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants