[Kimchi-devel] [PATCH V5 0/4] check the python code with pyflakes

shaohef at linux.vnet.ibm.com shaohef at linux.vnet.ibm.com
Mon Mar 24 14:58:39 UTC 2014


From: ShaoHe Feng <shaohef at linux.vnet.ibm.com>

V4 -> V5
We do not need pyflakes to run or build Kimchi.
It is only used for developers to check code style. 

V3 -> V4
address Rodrigo's comment.
add delete the cloned template in rollback. 

V2 -> V3
address zhengsheng's comment.
improve the SKIP_PYFLAKES_ERR list.

V1 -> V2
address Crístian'comment:
remove the whole statement "info = self._get_vm(name).inf"
address ming's comment:
break down the patch into small patches by the pyflakes error.

Pyflakes analyzes programs and detects various errors. It works by
parsing the source file, not importing it, so it is safe to use on
modules with side effects. It's also much faster.

This is important to improve the code quality.


ShaoHe Feng (4):
  make pyflakes happly, remove the unused import module
  make pyflakes happly, remove unused availables
  add template_delete to rollback after create a template
  run pyflakes when make check

 Makefile.am                      | 7 +++++++
 configure.ac                     | 7 +++++++
 src/kimchi/control/plugins.py    | 2 +-
 src/kimchi/featuretests.py       | 2 --
 src/kimchi/mockmodel.py          | 5 ++---
 src/kimchi/model/storagepools.py | 2 +-
 src/kimchi/screenshot.py         | 1 -
 src/kimchi/template.py           | 1 -
 tests/test_model.py              | 2 ++
 tests/test_server.py             | 1 -
 ui/pages/help/gen-index.py       | 1 -
 11 files changed, 20 insertions(+), 11 deletions(-)

-- 
1.8.5.3




More information about the Kimchi-devel mailing list