search

tfug / proofreading

https://github.com/tensorflow/docs の翻訳作業の校正ツールです！
7 stars 4 forks source link

add function to check 1file #11

Closed chie8842 closed 5 years ago

chie8842 commented 5 years ago

9 のPR作りました。

ただ、本当はファイル名を引数で与えたいと思ったのですが、makeコマンドは引数を受け取れない仕様なので、一旦1ファイルだけテストしたいときはファイル名をMakefileに記入するようにしました。

他の代替案としては、Makefileを使わない運用にして、シェルだけで全て解決するようにする、環境変数でファイル名を与えるなどが考えられます。

sfujiwara commented 5 years ago

あー、悩ましいですね。

Makefile あんまり詳しくないので適当に調べてきたやつだけど、こんな感じので何とかなったりしませぬ？

run-single-check:
    echo ${file}
make file=hoge.ipynb run-single-check
chie8842 commented 5 years ago

Makefileは引数を全てサブコマンドとして認識してしまうのでできないんですよね。。 Makefileでないと実現できない複雑なことをさせているわけでもないし、移植は簡単なので、 Makefile使わずに、./run filename みたいなかんじで今と同じことが実現できるようにするのがいいかなと思いました。 せっかく覚えたmakeコマンドが使えなくなるのは最初ちょっとめんどくさいと思われるかもしれませんが。。

sfujiwara commented 5 years ago

Makefile使わずに、./run filename みたいなかんじで今と同じことが実現できるようにするのがいいかなと思いました。

これでも OK ですー 👍

chie8842 commented 5 years ago

お、じゃあそうしますね！

chie8842 commented 5 years ago

bin/run-checkとかで実行するようにしてみました！

sfujiwara commented 5 years ago

あ〜 #10 をマージしたからコンフリクトが発生したっぽいですね ちょいと僕の方でもどう直すと良さそうか見てみますが、サクッと直せそうだったら直して貰えると 🙏

chie8842 commented 5 years ago

なおしましたー！