[blivet:master 0/2] Add a new pylint checker (v2)

mulhern amulhern at redhat.com
Fri Nov 21 18:07:32 UTC 2014


The changes from previous version are:
* Fixed a bug in the checker where it would give false positive in case
where there is an overridden value that is not a literal. There were no
instances where this was occurring in the code, so it was not an obvious
problem.
* Commented the checker analysis more heavily to document its restrictions.
* Added an additional checker in the same vein that detects a tiny subset of
pointless function overrides.
* Removed the (1) pointless function override that the new checker found
(in second patch).

mulhern (2):
  Add a simple pylint checker for pointless overrides.
  Remove pointless overrides.

 blivet/formats/biosboot.py         |   1 -
 blivet/formats/disklabel.py        |   1 -
 blivet/formats/dmraid.py           |   3 -
 blivet/formats/fs.py               |  10 --
 blivet/formats/luks.py             |   1 -
 blivet/formats/multipath.py        |   3 -
 blivet/formats/prepboot.py         |   1 -
 tests/formats_test/init_test.py    |   3 -
 tests/pylint/pointless-override.py | 194 +++++++++++++++++++++++++++++++++++++
 tests/pylint/pylint-one.sh         |   2 +-
 10 files changed, 195 insertions(+), 24 deletions(-)
 create mode 100644 tests/pylint/pointless-override.py

-- 
1.9.3



More information about the anaconda-patches mailing list