-
Notifications
You must be signed in to change notification settings - Fork 12
/
.ruff.toml
197 lines (134 loc) · 4.12 KB
/
.ruff.toml
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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
target-version = "py38"
exclude = [
]
lint.select = [
# ruff's default set of rules: https://docs.astral.sh/ruff/configuration/#using-pyprojecttoml
#"E4", "E7", "E9", "F",
# pyflakes
"F",
# pycodestyle
"E",
"W",
# mccabe
#"C90", # disabled, complexity is ok
# isort
"I",
# pep8-naming
"N",
# pydocstyle
#"D", # disabled, too pedantic, docs are not generated
# pyupgrade
"UP",
# flake8-2020
"YTT",
# flake8-annotations
#"ANN", # disabled, type annotation not used
# flake8-async
#"ASYNC", # disabled, aync not used
# flake8-bandit
#"S", # disabled, too many false positives
# flake8-blind-except
#"BLE", # disabled, these are typically the last 'catch-all' handler for anything not caught higher up.
# flake8-boolean-trap
#"FBT", # disabled, these aren't *that* confusing, c'mon.
# flake-8-bugbear
"B",
# flake8-builtins
"A",
# flake8-commas
#"COM", # disabled, inserts trailing commas into function calls, too weird.
# flake8-copyright
#"CPY", # disabled, too onerous
# flake8-comprehensions
"C4",
# flake8-datetimez
"DTZ",
# flake8-debugger
"T10",
# flake8-django
# "DJ" # disabled, django not used
# flake8-errmsg
"EM",
# flake8-executable
"EXE",
# flake8-future-annotations
#"FA", # disabled, type annotations not used
# flake8-implicit-str-concat
"ISC",
# flake8-import-conventions
"ICN", # disabled, too opinionated
# flake8-logging-format
"G",
# flake8-no-pep420
#"INP", # todo
# flake8-pie
"PIE",
# flake8-print
# "T20", # disabled, nothing wrong with print statements
# flake8-pyi
#"PYI", # disabled, type annotations not used
# flake8-pytest-style
#"PT", # todo
#flake8-quotes
#"Q", # todo
# flake8-raise
#"RSE", # disabled, too opinionated
# flake8-return
"RET",
# flake8-self
#"SLF", # disabled, public/private separation not really followed in builder
# flake8-slots
#"SLOT", todo
# flake8-simplify
"SIM",
# flake8-tidy-imports
"TID",
# flake8-type-checking
#"TCH", # disabled, type annotations not used
# flake8-gettext
"INT",
# flake8-unused-arguments
#"ARG", # todo
# flake8-use-pathlib
#"PTH", # disabled, pathlib not used
# flake8-todos
#"TD", # todo ;)
# flake8-fixme
# "FIX", # disabled, enabling would prevent code with TODOs from passing CI. not practical.
# eradicate
#"ERA", # todo
# pandas-vet
#"PD", # disabled, pandas not used
# pygrep-hooks
"PGH",
# pylint
"PL",
# tryceratops
#"TRY", # todo
"FLY",
# NumPy-specific rules
# "NPY" # disabled, numpy not used
# Airflow
# "AIR", # disabled, airflow not used
# Perflint
"PERF",
# refurb
#"FURB", # disabled, still in preview
# flake8-logging
# "LOG", # disabled, still in preview
# ruff
"RUF",
]
lint.ignore = [
"RUF005", # "Consider {expression} instead of concatenation". solution is unreadable and optimised for speed.
"RUF012", # "Mutable class attributes should be annotated with `typing.ClassVar`". # code is not type annotated
"E501", # "Line too long". long lines are ok, contortions to obey rule are not ok.
"E731", # "Do not assign a `lambda` expression, use a `def`". lambdas are fine.
"UP031", # "Use format specifiers instead of percent format". percent formatters are fine. f-strings absorb the environment.
"PLR0913", # "Too many arguments in function definition". complexity is ok.
"PLR0915", # "Too many statements". complexity is ok.
"PLR0912", # "Too many branches". complexity is ok
"PIE804", # "Unnecessary dict kwargs", boto often requires many long parameter names. dicts are nicer in this case.
"RET504", # "Unnecessary assignment to {name} before return statement". typically deliberate for readability/debugging/extensibilty.
"TRY300", # "Consider moving this statement to an else block", might have merit but syntax is strange and unfamiliar
]