search

RPMTW / RPMTW-Discord-Bot-python

RPMTW Discord Bot but write with Python
GNU General Public License v3.0
0 stars 0 forks source link

refactor: fix code style #3

Closed a3510377 closed 1 year ago

a3510377 commented 1 year ago

refactor code style

EvanHsieh0415 commented 1 year ago

refactor跟fix請分兩個pr

a3510377 commented 1 year ago

fix 也就加幾個字而已好嘛ㄏㄨㄢ呢 :D

Euxcbsks commented 1 year ago

抱歉，但reject

Euxcbsks commented 1 year ago

請把TYPE_CHECKING放回來

Euxcbsks commented 1 year ago

另外，系統訊息的Bug目前還沒有打算要修 我想先確定頻道是否真的要允許開啟討論串

EvanHsieh0415 commented 1 year ago

refactor真的最好跟其他分開來 不然太亂了

a3510377 commented 1 year ago

移除對 系統訊息的Bug 修復 而關於 TYPE_CHECKING，TYPE_CHECKING 基本是對於循環導入錯誤的解決，我的寫法並不會造成以上情況，並且可加快開發進程減少程式的重複所以我覺得這是可以的，故不會進行更改，若還是覺得有誤，可直接移除該 PR

Euxcbsks commented 1 year ago

這邊使用TYPE_CHECKING的目的是減少運行時import, 不是為了解決循環import

EvanHsieh0415 commented 1 year ago

然後title變得更縮略了

EvanHsieh0415 commented 1 year ago

先推遲

a3510377 commented 1 year ago

https://peps.python.org/pep-0563/#runtime-annotation-resolution-and-type-checking

Euxcbsks commented 1 year ago

https://peps.python.org/pep-0563/#runtime-annotation-resolution-and-type-checking

這並不足以成為接受的理由，因為這是已在考慮範圍內的影響