Merge pull request #829 from sirosen/pylint-require-param-docs

Move pylint config into setup.cfg and cleanup

setup.cfg

@@ -106,3 +106,40 @@ categories =
     Documentation
     Security
     Development
+
+
+[pylint]
+load-plugins = pylint.extensions.docparams
+
+# accept-no-param-doc makes docparams behave as desired/expected and flag any
+# method with missing param docs
+# toggle this on and off manually until we can get the number of failures
+# to 0, then leave it on
+#
+# accept-no-param-doc = false
+[pylint.messages control]
+disable =
+    # formatting and cosmetic rules (handled by 'black', etc)
+    format, C,
+    # refactoring rules (e.g. duplicate or similar code) are very prone to
+    # false positives
+    R,
+    # emitted when pylint fails to import a module; these warnings
+    # are usually false-positives for optional dependencies
+    import-error,
+    # "disallowed" usage of our own classes and objects gets underfoot
+    protected-access,
+    # logging-*-interpolation
+    # objections to log messages doing eager (vs lazy) string formatting
+    # this is potentially an area for improvement, but the benefit of deferred
+    # logging doesn't always outweigh the readability cost
+    logging-fstring-interpolation,
+    logging-format-interpolation,
+    # fixme comments are often useful; re-enable this to quickly find FIXME and
+    # TODO comments
+    fixme,
+    # most SDK methods currently do not document the exceptions which they raise
+    # this is an area for potential improvement
+    missing-raises-doc,
+[pylint.variables]
+ignored-argument-names = args|kwargs