Takashi Hashida
Takashi Hashida
Is this a work in progress?
@kou すみません、日本語の段階で一度内容に問題ないかどうかレビューを頂けないでしょうか？ 問題なさそうであれば翻訳作業に移ります。
@kou 指摘事項について対応しました。再度確認をお願いいたします。
レビューありがとうございます。 一旦翻訳に移ろうと思います。 まだなにかありましたら翻訳後の内容に頂ければと思います。
Cc: @kou 翻訳しましたので、再度確認頂けないでしょうか？
> 1点質問ですが、 parameterをセットするときは一つのコマンドと対になるものでしょうか？ > 文章中で何か所か this parameterと言っていたのですが、その手前にコマンドが何個も並んでいる場合は theseのような気もしました。 こちらの件ですが、n_workersというパラメータはselectコマンドというコマンドに対して一つでして、また、ここの記述は全体としてはn_workersにのみ言及しているので、thisで良いと考えています。
指摘事項に対応したものをpushしました。 一旦「valid」については「enabled」に変更しています。「supported」の方がふさわしければ再度対応します。
> 「このパラメーターを使えば多段ドリルダウンできます」とかもっとシンプルな言い回しにしませんか？ そのように修正しました。
@yoshimotoyuk Would you re-review English for this patch when you have time?
@yoshimotoyuk Thank you! @kou Would you re-review this when you have time?