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

Translate contribute/plugin-patterns.md #179

Merged
merged 14 commits into from
Sep 8, 2021

Conversation

yoonminsohappy
Copy link
Contributor

Summary

#7 (comment)

@yoonminsohappy yoonminsohappy requested a review from a team as a code owner August 17, 2021 01:27
@choilim
Copy link
Contributor

choilim commented Aug 17, 2021

Assignees 지정 부탁드려요~

@dkstyle dkstyle changed the title add: kr translation Translate contribute/plugin-patterns.md Aug 17, 2021
@dkstyle
Copy link
Contributor

dkstyle commented Aug 18, 2021

@yoonminsohappy
공식 문서 최신 내용 반영을 통해 plugin-patterns.md 파일이 plugin-patterns.mdx로 변경되었습니다.
kr upstream 브랜치를 fork 브랜치에 한번 머지한 후 수정사항과 함께 다시 확인 부탁드립니다.

@yoonminsohappy
Copy link
Contributor Author

yoonminsohappy commented Aug 18, 2021

공식 문서 최신 내용 반영을 통해 plugin-patterns.md 파일이 plugin-patterns.mdx로 변경되었습니다.
kr upstream 브랜치를 fork 브랜치에 한번 머지한 후 수정사항과 함께 다시 확인 부탁드립니다.

@dkstyle

  1. 코멘트 감사합니다! 리뷰 주신 부분 반영하였습니다.
  2. kr upstream branch 를 -> fork branch 에 머지하였습니다. 기존 .md 확장자 파일은 삭제하였습니다.
  3. (질문) line/webpack.kr 레포의 kr 브랜치fork 레포의 작업 브랜치 의 싱크를 맞추면 되는건가요?

제가 이해한 것이 맞는지 코멘트 부탁드립니다! 감사합니다!

@dkstyle
Copy link
Contributor

dkstyle commented Aug 18, 2021

공식 문서 최신 내용 반영을 통해 plugin-patterns.md 파일이 plugin-patterns.mdx로 변경되었습니다.
kr upstream 브랜치를 fork 브랜치에 한번 머지한 후 수정사항과 함께 다시 확인 부탁드립니다.

@dkstyle

  1. 코멘트 감사합니다! 리뷰 주신 부분 반영하였습니다.
  2. kr upstream branch 를 -> fork branch 에 머지하였습니다. 기존 .md 확장자 파일은 삭제하였습니다.
  3. (질문) line/webpack.kr 레포의 kr 브랜치fork 레포의 작업 브랜치 의 싱크를 맞추면 되는건가요?

제가 이해한 것이 맞는지 코멘트 부탁드립니다! 감사합니다!

@yoonminsohappy 현재 line/webpack.kr의 kr브랜치 상태와 맞추어 주시면 됩니다. 잘하셨습니다 :)

md -> mdx file로 rename 된 것이기때문에, 별도로 삭제를 하지 않으셔도 kr브랜치 내용을 반영하셨다면, 올바르게 트래킹이 됩니다. 결과적으로는 md파일은 제거되고 mdx 파일만 남는것이 맞습니다.

최신 내용 반영 과정에서 컨텐츠가 추가되어서 추가로 번역 해야할 내용들이 있습니다. 확인 부탁드립니다.

@yoonminsohappy
Copy link
Contributor Author

@yoonminsohappy 현재 line/webpack.kr의 kr브랜치 상태와 맞추어 주시면 됩니다. 잘하셨습니다 :)

md -> mdx file로 rename 된 것이기때문에, 별도로 삭제를 하지 않으셔도 kr브랜치 내용을 반영하셨다면, 올바르게 트래킹이 됩니다. 결과적으로는 md파일은 제거되고 mdx 파일만 남는것이 맞습니다.

최신 내용 반영 과정에서 컨텐츠가 추가되어서 추가로 번역 해야할 내용들이 있습니다. 확인 부탁드립니다.

리뷰 감사드립니다!

fetch upstream & merge upstream 을 했습니다. 그런데 리뷰 주신 4가지 부분(추가 컨텐츠 번역, 누락된 부분 번역)이 확인되지 않는 걸 보니, 올바르게 반영되지 않은 것 같습니다.

이 4가지 부분에 대한 반영은 조금 기다려주시면 감사하겠습니다! fork 브랜치가 최신화된 것인지 확인해 본 후 반영해보겠습니다.

@dkstyle dkstyle merged commit 7d338ad into line:kr Sep 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants