yuniruyuni commented 2 years ago

What is this?

各種requirements.txtのバージョンを(可能な限り)最新化ました
ライブラリアップデートに伴う必要なコード修正を行い動作する状態を保っています
ライブラリ自体がdeprecatedに近い状態になっているものについては別のライブラリに載せ替えました

Why did you create this PR?

Refer: #35 's discussion.
主にmaster branchを動作させて正しく動かない状態を脱却するためです。

How implemented this PR?

すべてのrequirements.txtのエントリでeditable modeは使わないようにした
すべてのrequirements.txtのエントリを一定バージョンで固定した
- Gemfile.lockやpackage-lock.jsonのような固定の仕組みがないため、勝手にmasterがビルド不能になる可能性があり、それを回避するため明示的に固定しました
- 今後のバージョン追従を考えるとPoetryやPipenvを導入する、dependabotを導入して監視させる、といった対処があると望ましいと思いますが一旦この形まで。
twitchioの2.x系APIインタフェイスに対応させた
- BOTの起動パラメタ設定を変えた
- decoratorだけで定義するモードが使えなくなったことに対処。Botクラスを継承する形に変更しました。
- contextの取り扱いの変更に対応。messageやchannel概念が正式に入りました
google_trans_newをasync_goolge_trans_newに載せ替えた
playsoundライブラリは1.2.2にしてあります。これはWIndows環境におけるplaysound1.3.0のバグを避けるためです:
- refer: https://stackoverflow.com/questions/68531326/what-is-the-error-in-the-code-for-this-playsound-module-even-though-the-syntax-i

Note

35 とconflictするのでmergeする場合、今見ているこのPR(#36)が先だとconflict解消が楽になる、と思います 🙇‍♀️

sayonari commented 2 years ago

これ今，ローカルにchecoutしてテストしてみてるんですが，requirements.txt の deepl==1.8.0 って，翻訳されますか？　今テストしたら翻訳されなくて，deepl だけで指定すると，翻訳できないパッケージがインストールされたので，前のrequirementsでは，翻訳できるver（PyPIには無くて，PyPIのdeeplと名前が被ってるやつ）をURL指定でインストールするようにしていました．今，ゆにるゆに先生が指定されてるやつ（PyPIからインストールされるやつ）で，そちらで　deeplが動作してますか？？

sayonari commented 2 years ago

今まで僕が使ってたパッケージが以下のURLのもので，

https://github.com/ptrstn/deepl-translate

今はPyPIにdeepl-translateで登録されているようです．

$ pip install deepl-translate

最新版だと 1.2.0 がインストールされ，それでこちらの環境（MacOS）でも動いてます． requirements.txt に，

deepl-translate==1.2.0

として対処しようと思います！！！！

sayonari / twitchTransFreeNext

update all dependencies #36

What is this?

Why did you create this PR?

How implemented this PR?

Note

35 とconflictするのでmergeする場合、今見ているこのPR(#36)が先だとconflict解消が楽になる、と思います 🙇‍♀️