-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
ci: make lint flake8 independent step #1507
Conversation
Latency summaryCurrent PR yields:
Breakdown
Backed by latency-tracking. Further commits will update this comment. |
b63aa28
to
435aa7f
Compare
Codecov Report
@@ Coverage Diff @@
## master #1507 +/- ##
==========================================
+ Coverage 84.16% 84.44% +0.27%
==========================================
Files 106 106
Lines 6118 6183 +65
==========================================
+ Hits 5149 5221 +72
+ Misses 969 962 -7
Continue to review full report at Codecov.
|
@hanxiao if accepted, how can I have the flake8 analysis as a required step? |
why separate it from the main ci jobs? |
no big deal, to have only one and not one per version |
No description provided.