Small refactor due to CI linter changes#3414
Conversation
|
Build succeeded.
|
|
The CI is greedy, it wants more |
|
"Don't feed the linters" |
Without the open variable in use, no reason to define it or use it as the for loop control. Signed-off-by: Phil Estes <[email protected]>
dmcgowan
force-pushed
the
refactor-forloop
branch
from
80a7fa8 to
82826df
Compare
|
Build succeeded.
|
Codecov Report
@@ Coverage Diff @@
## master #3414 +/- ##
=======================================
Coverage 45.05% 45.05%
=======================================
Files 113 113
Lines 12563 12563
=======================================
Hits 5660 5660
Misses 6049 6049
Partials 854 854
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #3414 +/- ##
=======================================
Coverage 45.05% 45.05%
=======================================
Files 113 113
Lines 12563 12563
=======================================
Hits 5660 5660
Misses 6049 6049
Partials 854 854
Continue to review full report at Codecov.
|
6 participants
Without the open variable in use, no reason to define it or use it as the
for loop control.
cc: @fuweid
Signed-off-by: Phil Estes [email protected]