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

Conditional Compilation #14

Merged
merged 7 commits into from
Jan 27, 2016
Merged

Conditional Compilation #14

merged 7 commits into from
Jan 27, 2016

Conversation

pocket7878
Copy link
Contributor

No description provided.

タイトルを翻訳しました
おおまかに翻訳してみました.
@pocket7878
Copy link
Contributor Author

featureというのが出てきたのですが,コレに関してはどのように訳せばよろしいでしょうか?
また,ソースコード中のコメントについても訳するべきでしょうか?
(# The “secure-password” feature depends on the bcrypt package. という箇所について)
@tatsuya6502, @KeenS 添削よろしくお願いします.

@KeenS
Copy link
Member

KeenS commented Jan 27, 2016

featureというのが出てきたのですが,コレに関してはどのように訳せばよろしいでしょうか?

featureも用語なので対訳表案件ですね。「フィーチャ」にしましょうか。相談なく勝手に訳を当てて対訳表も更新してプルリクを投げて頂ければ。問題のある時には議論するので。

ソースコード中のコメントについても訳するべきでしょうか?

はい、します。この時 #13 の問題があるので原文は保存しなくて大丈夫です。

添削よろしくお願いします.

上記までのコメントではなく訳文全体の話であれば [WIP] を先に取って下さい(つまり、翻訳作業が終わったら[WIP]を取って下さい)。

@pocket7878
Copy link
Contributor Author

@KeenS ありがとうございます.WIPの件了解しました.

タイトルを翻訳しました
おおまかに翻訳してみました.
対訳表にfeatureをフィーチャとして追加
`[features]`は設定ファイルの中の値なので訳さない方に戻した
@pocket7878 pocket7878 changed the title [WIP] Conditional Compilation Conditional Compilation Jan 27, 2016
@KeenS
Copy link
Member

KeenS commented Jan 27, 2016

良さそうです。ありがとうございました。

KeenS added a commit that referenced this pull request Jan 27, 2016
@KeenS KeenS merged commit 7a3a3f7 into rust-lang-ja:master Jan 27, 2016
@KeenS
Copy link
Member

KeenS commented Jan 27, 2016

あ、レビューが甘かった。句読点が,.になってますね。マージしてしまったのでこちらで修正します。次回以降は,.ではなく句読点を使うようにして下さい。(,.を使いたいならイシューを立てて頂けると他の翻訳者と議論が出来ます。)

@pocket7878
Copy link
Contributor Author

ありがとうございます。
お手数をおかけしてしまって申し訳ありません。
以後気をつけます。

@pocket7878 pocket7878 deleted the conditional-compilation branch January 27, 2016 12:33
@tatsuya6502 tatsuya6502 modified the milestone: 1.6 Apr 28, 2016
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