Skip to content

Conversation

@wagurano
Copy link

다른 언어 번역에는 README-xxXX.md 형식으로 되어있어서 한국어도 README-koKO.md 로 바꾸면 어떤가요?

@neocoin
Copy link

neocoin commented Nov 29, 2014

Ruby 코어에는 표준이 없지만, language code와 country code 를 표현하시려면,
Korean, Korea => koKR 이 맞을겁니다. �

아래 링크 같은 표현을 참고해 볼수 있을것 같네요.

Country code => http://en.wikipedia.org/wiki/Country_codes:_J%E2%80%93K#.C2.A0South_Korea
Language code => http://en.wikipedia.org/wiki/List_of_ISO_639-1_codes

Java => http://blog.beany.co.kr/archives/644
Unix => https://wiki.archlinux.org/index.php/Locale_%28%ED%95%9C%EA%B5%AD%EC%96%B4%29

@wagurano
Copy link
Author

아하! 그러면 파일 이름도 수정해야도 해야하겠군요. README-koKR.md 로 링크와 파일이름 모두 조정해야겠네요.

한국어-한국을 나타내기 위해 koKR로 바꿉니다.
@dalzony
Copy link
Owner

dalzony commented Dec 1, 2014

@wagurano @neocoin
말씀하신 부분이 맞지만,
다른 언어와 일관성을 맞추기위해 파일명을 정한것이었습니다.
참고 링크 : rubocop#342

@dalzony dalzony closed this Dec 1, 2014
@wagurano
Copy link
Author

wagurano commented Dec 1, 2014

아하 그런 사연이 있었군요. 레일스 i18n 인터페이스 가이드를 보니 ko-KR 한국어-대한민국 규칙이 더 어울려 보입니다. en-US en-GB 처럼 뒤에는 코드는 국가가 맞는데 KO라는 국가 코드는 사용하지 않아 보입니다. 원 저자에게 영어로 이야기 해보겠습니다.

@neocoin
Copy link

neocoin commented Dec 1, 2014

@dalzony 해당 쓰레드를 봐도, 한국 코드를 모르는 원저자를 가이드 해야죠.
원저자가 iso 크게 중시않고, koKO 로 했는데, 의도가 들어가려면, koKR 이 맞습니다.
http://en.wikipedia.org/wiki/ISO_3166-1_alpha-2

https://github.com/bbatsov/ruby-style-guide 가이드의 중국어 번역링크를 참고하세요.

@dalzony
Copy link
Owner

dalzony commented Dec 2, 2014

@wagurano @neocoin
예 😃 , 저 또한 iso를 별로 생각해보지 않아서 원저자 의견을 따랐습니다.
의뢰해서 고치도록 하겠습니다.

감사합니다 👍

아울러 몇달전 초기 번역 이후
원 글도 수정이 많이 되었습니다. (#5)
관심가져주세요.

@dalzony dalzony reopened this Dec 2, 2014
@dalzony dalzony merged commit 0cd74e0 into dalzony:master Dec 8, 2014
dalzony added a commit that referenced this pull request Dec 8, 2014
@dalzony
Copy link
Owner

dalzony commented Dec 8, 2014

링크 수정해서, 원글에도 반영시켰습니다. @wagurano @neocoin

dalzony added a commit that referenced this pull request Apr 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants