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
Add pytest-flake8 #3945
Add pytest-flake8 #3945
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3945 +/- ##
=========================================
- Coverage 84.96% 84.4% -0.56%
=========================================
Files 166 166
Lines 9681 9681
Branches 1445 1392 -53
=========================================
- Hits 8225 8171 -54
- Misses 1194 1233 +39
- Partials 262 277 +15
|
Codecov Report
@@ Coverage Diff @@
## master #3945 +/- ##
==========================================
+ Coverage 83.36% 83.37% +0.01%
==========================================
Files 165 165
Lines 9802 9802
Branches 1462 1462
==========================================
+ Hits 8171 8172 +1
Misses 1366 1366
+ Partials 265 264 -1
|
6a2aac0
to
697168a
Compare
Seems that the pipeline worked: https://travis-ci.org/scrapy/scrapy/jobs/569955789 |
@noviluni Looks great. However be ready for something like #3727 (comment) from @dangra :) |
Hi @Gallaecio thank you for your feedback. I've been thinking that, and I'm not totally agree (in fact, I can't fully understand the reasons)... The idea here is to avoid big PRs to be reviewed and adding As my original proposal was to avoid that, for me it has no sense. I can't see the benefits of adding all those lines and then delete them when fixing the code. It involves more work (twice) and probably more "risk" (looking at your PR: 170 files to review is kind of risky). @dangra what do you think? Is it necessary to mark all the cases one by one and then start fixing them or could we start this way and when fixing every flake8 rule add the necessary Thanks in advance |
697168a
to
9e9ee68
Compare
9e9ee68
to
05a4adb
Compare
Updated to use a list files (with the excluded rules) instead of ignoring all rules directly. In that way it's possible to disable checks only for the files that are not passing them. |
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.
I’ve left a couple of comments, but I’m OK with merging this as is.
thank you for your feedback 😄 |
proposal for: #3944
Please, take in mind that it’s only a PoC. In this way the testing time increases and maybe it’s not necessary to check flake8 against all environments.