088d57870c
This is a follow-up from [1] which underlined that the code lacks even basic coverage as it allow
us to introduce a costly regression which could have being detected by pylint no-member test.
- enables pylint with most checks temporary disabled, so we have time
to address them gradually
- fixes few minor issues reported by the tool
- adds skips for some known "no-member" errors but avoids adding it to the exclude list, as this
in order to prevent further regressions.
- once landed we can easily address the temporary disabled errors, one by one. In fact this is could
prove as a very good learning experience for newer team members. They can start by removing on
random exclusion and fixing it. As a hint: leave the missing docstrings for the end, some problems
are more important to fix first.
1: https://review.opendev.org/c/openstack/tripleo-common/+/762892/6/tripleo_common/image/builder/buildah.py
Change-Id: I10ab0cbfbaab77b9208e9a5d74d59eb041cb16ee
(cherry picked from commit
|
||
---|---|---|
.. | ||
__init__.py | ||
base.py | ||
processlock.py | ||
threadinglock.py |