-
Notifications
You must be signed in to change notification settings - Fork 127
wheelを利用したexampleをわかりやすく #421
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
改良ありがとうございます、助かります!!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@misogihagi 結構wheel版のPythonサンプルを通ろうとして、同じようにREADMEの不足で困られている方が多い印象です。
頂いているPRでREADMEをしっかり整備してみませんか・・・! 👀
あとは別PRで頂いたDownloader周りもちょっと追従して、wheelのURLの案内辺りを整理すれば完成かなと思っています!!
#446 の件はもともと抱えていたものでどうしようもない気はしますが… |
@misogihagi たしかにそうですね、書き足してみました! |
@misogihagi すみません、コンフリクト解消はこちらでしてしまいました! |
@misogihagi お忙しいようでしたらこのPRも後はこちらでやらせて頂こうかなと思っています。いかがでしょうか? |
PR開発者の方からOKをいただくか、数日待って返信がないなどであれば大丈夫です・・・! |
@qryxip |
コンフリクト解消はしましたが、#475で"open-jtalk-dict-dir"のデフォルト値が |
いや
の両方が同時に無いといけなくて、今のreadmeだとそうなっていない。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
たぶんほぼLGTMです!!
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
Co-authored-by: Hiroshiba <hihokaruta@gmail.com>
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
実際にREADMEに従ってlinux環境で試せることも確認しました! 良いですね!!
テスト待ちです、テスト通ったらマージしていただけると。 @qryxip 引き継ぎありがとうございました!! |
テスト通ったのでマージしてしまいます。 @misogihagi さんと、途中から引き継いだ @qryxip さん感謝です……! |
内容
wheelを利用するとさらの状態からの実行がやりやすくなりました。
その手順がイマイチわかりにくかったのでコマンドを添えておきました。
関連 Issue
#414