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

Update ja/kicad.po for 5.0.0 #33

Closed
wants to merge 1 commit into from

Conversation

asukiaaa
Copy link
Member

@asukiaaa asukiaaa commented Apr 1, 2018

ja/kicas.po の翻訳を行いました。
#30 では先頭の20を翻訳すると宣言していましたが、ファイル内の未翻訳となっている箇所全ての翻訳を行いました。

確認をお願いします。

@kinichiro
Copy link

@asukiaaa さん
GUIの翻訳作業ありがとうございます。チェックしてみました。
'&'の扱いなどもきちんとされていますし、訳文も分かり易かったです。
とても良い仕事だと思いました。
また PoEdit で .mo が作成されることも確認できました。

1点だけ、
ID.5750 "Executing '%s'" を「除外 '%s'」とされてますが、これは動作確認してみてのことでしょうか?
実行とか実行中の方が妥当かなと感じましたが、いかがでしょう。

それからこれは翻訳の内容の話ではありませんが、PRやコミットのタイトルは英語で書いたほうが、本家にコミットされた時に外国のコミッターの人にも概要が分かってよいと思います。

私からは以上です。

@asukiaaa asukiaaa changed the title 日本語ファイルを翻訳 Translate ja/kicad.po Apr 1, 2018
@asukiaaa asukiaaa force-pushed the feature/translate-ja branch from 974d5b2 to 5322b7d Compare April 1, 2018 06:27
@asukiaaa asukiaaa changed the title Translate ja/kicad.po Update ja/kicad.po for 5.0.0 Apr 1, 2018
@asukiaaa
Copy link
Member Author

asukiaaa commented Apr 1, 2018

@kinichiro さん
丁寧なチェックとコメントありがとうございます。

ID.5750 "Executing '%s'" を「除外 '%s'」とされてますが、これは動作確認してみてのことでしょうか?
実行とか実行中の方が妥当かなと感じましたが、いかがでしょう。

指摘ありがとうございます。
Excludeと読み間違えていました。
「実行中」に訂正しました。

それからこれは翻訳の内容の話ではありませんが、PRやコミットのタイトルは英語で書いたほうが、本家にコミットされた時に外国のコミッターの人にも概要が分かってよいと思います。

アドバイスありがとうございます。
5.0.0向けの更新と分かるように、PRとコミットのタイトルを変更しました。

確認をお願いします。

@kinichiro
Copy link

kinichiro commented Apr 1, 2018 via email

@asukiaaa
Copy link
Member Author

asukiaaa commented Apr 1, 2018

確認ありがとうございます。
指摘が無ければ、後のことをよろしくお願いします。

@asukiaaa
Copy link
Member Author

asukiaaa commented Apr 1, 2018

別のPRが作成されていたので、クローズします。
KiCad#233

@asukiaaa asukiaaa closed this Apr 1, 2018
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.

2 participants