hardbyte / hardbyte/python-can

Enable remaining Pylint rules

Đang mở
#613 1 bình luận 1 reaction 0 người được giao Xem trên GitHub
QA
Ngôn ngữ chính
Python
Star
1.6k
Fork
697
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.