hardbyte / hardbyte/python-can

Enable remaining Pylint rules

Aberta
#613 1 comentário 1 reação 0 responsáveis Ver no GitHub
QA
Linguagem predominante
Python
Estrelas
1.6k
Forks
697
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Currently, the Pylint rules disable the following checks (see [`/.pylintrc-wip`](https://github.com/hardbyte/python-can/blob/a7a9c096134b6fb0b2308c870bb379c6e5d8f4f6/.pylintrc-wip#L63)):

```
disable=invalid-name,
missing-docstring,
empty-docstring,
line-too-long,
too-many-lines,
bad-whitespace,
bad-continuation,
wrong-import-order,
ungrouped-imports,
wrong-import-position,
too-many-function-args,
locally-disabled,
too-many-instance-attributes,
too-few-public-methods,
too-many-public-methods,
too-many-branches,
too-many-arguments,
too-many-locals,
too-many-statements,
too-many-nested-blocks,
no-else-raise,
bad-indentation,
wildcard-import,
fixme,
broad-except,
redefined-builtin,
no-else-return,
redefined-argument-from-local,
abstract-class-instantiated,
multiple-statements,
cyclic-import,
useless-else-on-loop,
duplicate-code,
inconsistent-return-statements,
arguments-differ,
unused-wildcard-import,
logging-fstring-interpolation,
logging-format-interpolation,
unused-argument,
abstract-method,
attribute-defined-outside-init,
protected-access,
keyword-arg-before-vararg
```

Do we aim at eventually removing all of them? The default config does disable some rules as well (see [`/.pylintrc`](https://github.com/hardbyte/python-can/blob/a7a9c096134b6fb0b2308c870bb379c6e5d8f4f6/.pylintrc#L63))

#601 removed all previously disabled linter rules that did require only small changes.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.