John Snow <js...@redhat.com> writes: > This allows others to get repeatable results with pylint. If you run > `pylint iotests.py`, you should see a 100% pass.
Nice. > > Signed-off-by: John Snow <js...@redhat.com> > Reviewed-by: Max Reitz <mre...@redhat.com> > --- > tests/qemu-iotests/pylintrc | 22 ++++++++++++++++++++++ > 1 file changed, 22 insertions(+) > create mode 100644 tests/qemu-iotests/pylintrc > > diff --git a/tests/qemu-iotests/pylintrc b/tests/qemu-iotests/pylintrc > new file mode 100644 > index 0000000000..8720b6a0de > --- /dev/null > +++ b/tests/qemu-iotests/pylintrc > @@ -0,0 +1,22 @@ > +[MESSAGES CONTROL] > + > +# Disable the message, report, category or checker with the given id(s). You > +# can either give multiple identifiers separated by comma (,) or put this > +# option multiple times (only on the command line, not in the configuration > +# file where it should appear only once). You can also use "--disable=all" to > +# disable everything first and then reenable specific checks. For example, if > +# you want to run only the similarities checker, you can use "--disable=all > +# --enable=similarities". If you want to run only the classes checker, but > have > +# no Warning level messages displayed, use "--disable=all --enable=classes > +# --disable=W". > +disable=invalid-name, > + no-else-return, > + too-many-lines, > + too-few-public-methods, > + too-many-arguments, > + too-many-locals, > + too-many-branches, > + too-many-public-methods, Keep sorted? > + # These are temporary, and should be removed: > + missing-docstring, > + line-too-long, For what it's worth, I also disable these for checking QAPI code, except for no-else-return. My true reason for keeping no-else-return is of course that I agree with pylint these elses are ugly. But I pretend to simply go with the flow ;)
