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

TRANSLATION_GUIDE.md 용어 사용 규칙에 requirements 추가(#537) #541

Merged
merged 2 commits into from
Aug 25, 2022
Merged

Conversation

terri1102
Copy link
Contributor

라이선스 동의

변경해주시는 내용에 BSD 3항 라이선스가 적용됨을 동의해주셔야 합니다.

더 자세한 내용은 기여하기 문서를 참고해주세요.

동의하시면 아래 [ ][x]로 만들어주세요.

  • 기여하기 문서를 확인하였으며, 본 PR 내용에 BSD 3항 라이선스가 적용됨에 동의합니다.

관련 이슈 번호

이 Pull Request와 관련있는 이슈 번호를 적어주세요.

이슈 또는 PR 번호 앞에 #을 붙이시면 제목을 바로 확인하실 수 있습니다. (예. #999 )

PR 종류

이 PR에 해당되는 종류 앞의 [ ][x]로 변경해주세요.

  • 오탈자를 수정하거나 번역을 개선하는 기여
  • 번역되지 않은 튜토리얼을 번역하는 기여
  • 공식 튜토리얼 내용을 반영하는 기여
  • 위 종류에 포함되지 않는 기여

PR 설명

TRANSLATION_GUIDE 용어 사용 규칙에 requirements를 추가하였습니다.

@terri1102 terri1102 changed the title add: TRANSLATION_GUIDE.md 용어 사용 규칙에 requirements 추가(#537) TRANSLATION_GUIDE.md 용어 사용 규칙에 requirements 추가(#537) Aug 5, 2022
@hyoyoung hyoyoung added the 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR label Aug 5, 2022
Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

용어집 pr과 번역 pr은 분리해서 별도의 pr 2개로 보내주시기 바랍니다

@terri1102
Copy link
Contributor Author

같은 브랜치에서 작업하다가 꼬였네요;; 수정하겠습니다

@hyoyoung
Copy link
Member

컨트리뷰톤 회의에서 merge 결정하였습니다

@hyoyoung
Copy link
Member

@terri1102
이름 옆의 칸에 음차표기 라고 적어주시고
re-request 버튼 눌러주세요

@terri1102
Copy link
Contributor Author

@hyoyoung requirements를 요구 사항으로 번역하는데 음차표기라고 적어야 할까요?

@hyoyoung
Copy link
Member

@hyoyoung requirements를 요구 사항으로 번역하는데 음차표기라고 적어야 할까요?

오 아녜요 이건 제가 실수했네요. 다른거하고 착각했습니다. 그냥 두시고 re-request review버튼은 꼭 눌러주세요

Copy link
Member

@hyoyoung hyoyoung left a comment

Choose a reason for hiding this comment

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

good

@hyoyoung hyoyoung merged commit e75aece into PyTorchKorea:master Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
용어집 컨트리뷰톤 오픈소스 컨트리뷰톤 관련 이슈/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants