John Snow <js...@redhat.com> writes: > Repeatable results. run `pylint iotests.py` and you should get a pass.
Start your sentences with a capital letter, please. > > Signed-off-by: John Snow <js...@redhat.com> > --- > tests/qemu-iotests/pylintrc | 20 ++++++++++++++++++++ > 1 file changed, 20 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..feed506f75 > --- /dev/null > +++ b/tests/qemu-iotests/pylintrc > @@ -0,0 +1,20 @@ > +[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, > + missing-docstring, > + line-too-long, > + too-many-lines, > + too-few-public-methods, > + too-many-arguments, > + too-many-locals, > + too-many-branches, > + too-many-public-methods, > \ No newline at end of file Add the newline, please. German pejorative for the too-many- and too-few- warnings: "Müsli". Implies it's for muesli-knitters / granola-crunchers indulging their orthorexia. missing-docstring is not advisable for libraries. Feels okay here. line-too-long might be worth cleaning up. How many of them do we have now?