search

nonebot / aiocqhttp

A Python SDK with async I/O for CQHTTP (OneBot).
https://aiocqhttp.nonebot.dev
MIT License
280 stars 39 forks source link

修复 ActionFailed 不兼容问题 #54

Closed stdrc closed 3 years ago

stdrc commented 3 years ago

修复 #53

顺带改用 black 格式化了，之前 yapf 似乎不够强劲，有些地方不同风格它不会强制统一（比如单双引号）。

没有测试，请有兴趣的帮忙测试一下～

cleoold commented 3 years ago

hmm

是按照單引號格式化還是按照雙引號改的内容多？

Ice9Coffee commented 3 years ago

hmm

是按照單引號格式化還是按照雙引號改的内容多？

black默认是用双引号吧，除非字符串内含双引号 应当是把几乎所有的双引号字符串改成了单引号字符串

stdrc commented 3 years ago

是按照單引號格式化還是按照雙引號改的内容多？

black 默认双引号，而且好像不太好改？

另外就是改的内容多没什么，这又不是什么重要项目，以后全用双引号就可以了。改用 black 主要是它的默认配置比 yapf 更强劲，不需要折腾。

cleoold commented 3 years ago

我不喜歡雙引號，雙引號太醜了。況且我也不喜歡不支援任何自定義，含有獨裁意味的 black。把一大堆單引號改成雙引號好像除了讓文件記錄變長和讓風格變醜也沒有什麽作用。

stdrc commented 3 years ago

实际上在之前用 yapf 的时候有许多地方都没有格式化，而 flake8 的 lint 也没检查出来。

我还是希望用 black，定制性强确实好，但显然我们并没有好好定制，于是多种格式都会被 yapf 允许，换用 black 可以不再需要折腾这个。

以前我也觉得双引号丑，现在我觉得不错。每个人的审美都不同，代码风格并不是要让大家都觉得好看，而是让大家写出来的代码更统一。

cleoold commented 3 years ago

实际上在之前用 yapf 的时候有许多地方都没有格式化，而 flake8 的 lint 也没检查出来。

我还是希望用 black，定制性强确实好，但显然我们并没有好好定制，于是多种格式都会被 yapf 允许，换用 black 可以不再需要折腾这个。

以前我也觉得双引号丑，现在我觉得不错。每个人的审美都不同，代码风格并不是要让大家都觉得好看，而是让大家写出来的代码更统一。

這個倉庫和 nonebot/nonebot 統一就夠了。

linter 不止 black 這一個。可以先在這解決 retcode 的問題發佈 1.4.1，再討論之後的。

cleoold commented 3 years ago

taken over by https://github.com/nonebot/aiocqhttp/pull/55