-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
使用black做代码格式化 #1675
使用black做代码格式化 #1675
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1675 +/- ##
==========================================
- Coverage 76.20% 76.20% -0.01%
==========================================
Files 91 91
Lines 14306 14307 +1
==========================================
Hits 10902 10902
- Misses 3404 3405 +1
Continue to review full report at Codecov.
|
增加代码格式化检测
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.
Found 17 potential problems in the proposed changes. Check the Files changed tab for more details.
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.
Found 17 potential problems in the proposed changes. Check the Files changed tab for more details.
一万多行, 哈哈 |
最好加一个脚本方便本地跑 |
二开的合并的上么😂@nick2wang @weideguo |
|
看了下大部分是对单引号字符串格式化为双引号,black应该可以配置跳过这个检查项,毕竟在pep8规范里使用单引号是合规(不愧是一点都不妥协的black:joy:) |
感觉双引号可以有, 没必要忽略的样子 |
我没什么问题,可以使用主分支 |
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.
Found 17 potential problems in the proposed changes. Check the Files changed tab for more details.
action增加代码格式化检测