Closed DimitriPapadopoulos closed 6 months ago
Attention: Patch coverage is 21.27660%
with 37 lines
in your changes are missing coverage. Please review.
Project coverage is 70.90%. Comparing base (
d52a62d
) to head (7774e3f
).:exclamation: Current head 7774e3f differs from pull request most recent head cfcbc6d. Consider uploading reports for the commit cfcbc6d to get more accurate results
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary
Partially apply
FURB
rules, where they make sense, exceptFURB101
(26 errors),FURB103
(21 errors),FURB113
(2 errors),FURB118
(3 errors),FURB140
(2 errors),FURB171
(25 errors).List of changes proposed in this PR (pull-request)
list.extend(...)
instead of repeatedly callinglist.append()
readlines()
, iterate over file object directly20 if len(nodiffidx) >= 20 else len(nodiffidx)
withmin(20, len(nodiffidx))
enumerate
index is unused, usefor x in y
insteadre.X
FURB167 Use of regular expression aliasre.M