-
Notifications
You must be signed in to change notification settings - Fork 118
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
downloadクレートの名称をdownloaderにする #607
Comments
確かに、クレート名と本来の名前が合致していないので、変えちゃっていいのでは無いかなと思いました! |
よいと思います。 |
タイトル変えておきます! |
こちらやっちゃおうと思います! |
@qryxip @y-chan 実行バイナリの名称は、何をするかではなく、何をするものなのかを書いた方が良いと思ってます。 ただ、破壊的変更するべきもののほどなのかが若干自信が無く・・・。 まあでもわりとどっちでもいいと思うので、意見いただいたら素直にそっちに倒そうかなと思います! |
個人的にはバイナリ名は "builder.sh"のような名前より"build.sh"の方が多数ですし、Windows/Unix両方において、バイナリ名は"コマンド名"と捉えた方がよいのではと思っています。 |
なるほどです! 今ドキュメントでは |
|
内容
voicevox_coreクレートは、自身を意味する
voicevox_core
を名乗っていますが、downloaderはdownload
クレートになっています。現
download
クレートも自身を意味するdownloader
に変えたほうがわかりやすい気がしました。Pros 良くなる点
クレート内容とクレート名が同じになる。
補助的な効果として
cargo -p download
など初学者にとってちょっと分かりにくいコマンドじゃなくなる。Cons 悪くなる点
実現方法
クレート名を変更してREADMEなども変更。
その他
download
コマンドの名称をdownload
からdownloader
にすべきかはちょっと分かりませんでした。それでもいいと思うけれども変えるほどではないかも。
The text was updated successfully, but these errors were encountered: