[blivet:master 0/2] A couple of small pylint inspired patches.

mulhern amulhern at redhat.com
Tue Nov 18 23:09:08 UTC 2014


The first was inspired by a class of pylint errors, the C (Convention) ones
that are disabled in our tests. I happened to run pylint-gui for fun and this
one popped out at me by pure luck.

However, if the C error were undisabled there would be about 
3000 new errors, mostly quite pointless, about too-long-lines and missing-
docstrings.

The one that made me notice bug in patch (1) was the superfluous-parens
error. There are 55 other instances of just this, but they all appear benign.
Fixing them wouldn't be hard, but it would be a busy commit with no real
useful affect.

So I'm leaving the C errors disabled.

mulhern (2):
  Match whole string, not substring.
  Remove redundant import.

 blivet/arch.py     | 2 +-
 tests/size_test.py | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

-- 
1.9.3



More information about the anaconda-patches mailing list