forked from NVIDIA-Merlin/NVTabular
-
Notifications
You must be signed in to change notification settings - Fork 0
/
.pylintrc
73 lines (64 loc) · 1.83 KB
/
.pylintrc
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
[MASTER]
ignore-patterns=_version.py,model_config_pb2.py,versioneer.py,test_.*,setup.py
extension-pkg-allow-list=hugectr,nvtabular_cpp
[MESSAGES CONTROL]
disable=fixme,
# docstrings aren't required (yet).
missing-function-docstring,
missing-module-docstring,
missing-class-docstring,
# formatting checks that we're handling with black/isort
wrong-import-order,
wrong-import-position,
ungrouped-imports,
line-too-long,
superfluous-parens,
trailing-whitespace,
# we'll probably never enable these checks
invalid-name,
import-error,
# disable code-complexity checks for now
# TODO: should we configure the thresholds for these rather than just disable?
too-many-function-args,
too-many-instance-attributes,
too-many-locals,
too-many-branches,
too-many-nested-blocks,
too-many-statements,
too-many-arguments,
too-many-return-statements,
too-many-lines,
too-few-public-methods,
# many of these checks would be great to include at some point, but would
# require some changes to our codebase
useless-return,
protected-access,
arguments-differ,
unused-argument,
unused-variable,
abstract-method,
no-name-in-module,
attribute-defined-outside-init,
redefined-outer-name,
import-outside-toplevel,
no-else-continue,
no-else-return,
no-else-raise,
no-member,
super-with-arguments,
unsupported-assignment-operation,
inconsistent-return-statements,
duplicate-string-formatting-argument,
len-as-condition,
cyclic-import,
consider-using-f-string,
arguments-renamed,
# producing false positives
unexpected-keyword-arg,
not-an-iterable,
unsubscriptable-object
[SIMILARITIES]
min-similarity-lines=30
ignore-comments=yes
ignore-docstrings=yes
ignore-imports=yes