blob: 8447821b4e5554a721bc5acb3f85767b6311302e [file] [log] [blame]
Nathaniel Manistae151a212017-02-24 17:47:24 +00001[VARIABLES]
Nathaniel Manista91048842017-06-24 21:16:53 +00002
Nathaniel Manistae151a212017-02-24 17:47:24 +00003# TODO(https://github.com/PyCQA/pylint/issues/1345): How does the inspection
4# not include "unused_" and "ignored_" by default?
5dummy-variables-rgx=^ignored_|^unused_
6
Nathaniel Manistadb0f85b2017-03-03 05:41:50 +00007[DESIGN]
Nathaniel Manista91048842017-06-24 21:16:53 +00008
Nathaniel Manistadb0f85b2017-03-03 05:41:50 +00009# NOTE(nathaniel): Not particularly attached to this value; it just seems to
10# be what works for us at the moment (excepting the dead-code-walking Beta
11# API).
12max-args=6
13
Nathaniel Manista8903a192017-03-02 18:49:07 +000014[MISCELLANEOUS]
Nathaniel Manista91048842017-06-24 21:16:53 +000015
Nathaniel Manista8903a192017-03-02 18:49:07 +000016# NOTE(nathaniel): We are big fans of "TODO(<issue link>): " and
17# "NOTE(<username or issue link>): ". We do not allow "TODO:",
18# "TODO(<username>):", "FIXME:", or anything else.
19notes=FIXME,XXX
20
Mehrdad Afshari1a967c32017-02-13 11:38:17 -080021[MESSAGES CONTROL]
22
Nathaniel Manista91048842017-06-24 21:16:53 +000023disable=
24 # TODO(https://github.com/PyCQA/pylint/issues/59#issuecomment-283774279):
25 # Enable cyclic-import after a 1.7-or-later pylint release that
26 # recognizes our disable=cyclic-import suppressions.
27 cyclic-import,
28 # TODO(https://github.com/grpc/grpc/issues/8622): Enable this after the
29 # Beta API is removed.
30 duplicate-code,
31 # TODO(https://github.com/grpc/grpc/issues/261): Doesn't seem to
32 # understand enum and concurrent.futures; look into this later with the
33 # latest pylint version.
34 import-error,
35 # TODO(https://github.com/grpc/grpc/issues/261): Enable this one.
36 # Should take a little configuration but not much.
37 invalid-name,
38 # TODO(https://github.com/grpc/grpc/issues/261): This doesn't seem to
39 # work for now? Try with a later pylint?
40 locally-disabled,
41 # NOTE(nathaniel): We don't write doc strings for most private code
42 # elements.
43 missing-docstring,
44 # NOTE(nathaniel): Our completely abstract interface classes don't have
45 # constructors.
46 no-init,
47 # TODO(https://github.com/grpc/grpc/issues/261): Doesn't yet play
48 # nicely with some of our code being implemented in Cython. Maybe in a
49 # later version?
50 no-name-in-module,
51 # TODO(https://github.com/grpc/grpc/issues/261): Suppress these where
52 # the odd shape of the authentication portion of the API forces them on
53 # us and enable everywhere else.
54 protected-access,
55 # NOTE(nathaniel): Pylint and I will probably never agree on this.
56 too-few-public-methods,
57 # NOTE(nathaniel): Pylint and I wil probably never agree on this for
58 # private classes. For public classes maybe?
59 too-many-instance-attributes,
60 # NOTE(nathaniel): Some of our modules have a lot of lines... of
61 # specification and documentation. Maybe if this were
62 # lines-of-code-based we would use it.
63 too-many-lines,
64 # TODO(https://github.com/grpc/grpc/issues/261): Maybe we could have
65 # this one if we extracted just a few more helper functions...
66 too-many-nested-blocks,
67 # NOTE(nathaniel): I have disputed the premise of this inspection from
68 # the beginning and will continue to do so until it goes away for good.
69 useless-else-on-loop,