Re: [PATCH 1/6] python: disable pylint errors for aqmp-tui

[John Snow <jsnow@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1119 bytes --]

On Fri, Jul 2, 2021 at 5:26 PM G S Niteesh Babu <niteesh.gs@gmail.com>
wrote:

> Disable missing-docstring and fixme pylint warnings.
> This is because since the AQMP is just a prototype
> it is currently not documented properly and lot
> of todo and fixme's are still in place.
>
> Signed-off-by: G S Niteesh Babu <niteesh.gs@gmail.com>
> ---
>  python/setup.cfg | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/python/setup.cfg b/python/setup.cfg
> index bce8807702..1a552d672a 100644
> --- a/python/setup.cfg
> +++ b/python/setup.cfg
> @@ -89,6 +89,8 @@ ignore_missing_imports = True
>  # no Warning level messages displayed, use "--disable=all --enable=classes
>  # --disable=W".
>  disable=too-many-function-args,  # mypy handles this with less false
> positives.
> +        missing-docstring, # FIXME
> +        fixme, # FIXME
>

You aren't actually using any FIXME statements in this branch right now
that I can see, so you don't need that suppression. It could be removed in
V3.


>
>  [pylint.basic]
>  # Good variable names which should always be accepted, separated by a
> comma.
> --
> 2.17.1
>
>

[-- Attachment #2: Type: text/html, Size: 1776 bytes --]