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
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
|
from argparse import ArgumentParser
from argparse import RawDescriptionHelpFormatter
from collections.abc import Iterator
from pathlib import Path
import re
from black import DEFAULT_LINE_LENGTH
from black import format_str
from black import Mode
from black import parse_pyproject_toml
from black import TargetVersion
home = Path(__file__).parent.parent
_Block = list[tuple[str, int, str | None, str]]
def _format_block(
input_block: _Block, exit_on_error: bool, is_doctest: bool
) -> list[str]:
code = "\n".join(c for *_, c in input_block)
try:
formatted = format_str(code, mode=BLACK_MODE)
except Exception as e:
if is_doctest:
start_line = input_block[0][1]
print(
"Could not format code block starting at "
f"line {start_line}:\n{code}\nError: {e}"
)
if exit_on_error:
print("Exiting since --exit-on-error was passed")
raise
else:
print("Ignoring error")
elif VERBOSE:
start_line = input_block[0][1]
print(
"Could not format code block starting at "
f"line {start_line}:\n---\n{code}\n---Error: {e}"
)
return [line for line, *_ in input_block]
else:
formatted_code_lines = formatted.splitlines()
padding = input_block[0][2]
if is_doctest:
formatted_lines = [
f"{padding}>>> {formatted_code_lines[0]}",
*(f"{padding}... {fcl}" for fcl in formatted_code_lines[1:]),
]
else:
# The first line may have additional padding.
# If it does restore it
additionalPadding = re.match(
r"^(\s*)[^ ]?", input_block[0][3]
).groups()[0]
formatted_lines = [
f"{padding}{additionalPadding}{fcl}" if fcl else fcl
for fcl in formatted_code_lines
]
if not input_block[-1][0] and formatted_lines[-1]:
# last line was empty and black removed it. restore it
formatted_lines.append("")
return formatted_lines
doctest_code_start = re.compile(r"^(\s+)>>>\s?(.+)")
doctest_code_continue = re.compile(r"^\s+\.\.\.\s?(\s*.*)")
plain_indent = re.compile(r"^(\s{4})(\s*[^: ].*)")
format_directive = re.compile(r"^\.\.\s*format\s*:\s*(on|off)\s*$")
dont_format_under_directive = re.compile(r"^\.\. (?:toctree)::\s*$")
def format_file(
file: Path, exit_on_error: bool, check: bool, no_plain: bool
) -> bool | None:
buffer = []
if not check:
print(f"Running file {file} ..", end="")
original = file.read_text("utf-8")
doctest_block: _Block | None = None
plain_block: _Block | None = None
last_line = None
disable_format = False
non_code_directive = False
for line_no, line in enumerate(original.splitlines(), 1):
if match := format_directive.match(line):
disable_format = match.groups()[0] == "off"
elif match := dont_format_under_directive.match(line):
non_code_directive = True
if doctest_block:
assert not plain_block
if match := doctest_code_continue.match(line):
doctest_block.append((line, line_no, None, match.groups()[0]))
continue
else:
buffer.extend(
_format_block(
doctest_block, exit_on_error, is_doctest=True
)
)
doctest_block = None
if plain_block:
assert not doctest_block
if not line:
plain_block.append((line, line_no, None, line))
continue
elif match := plain_indent.match(line):
plain_block.append((line, line_no, None, match.groups()[1]))
continue
else:
if non_code_directive:
buffer.extend(line for line, _, _, _ in plain_block)
else:
buffer.extend(
_format_block(
plain_block, exit_on_error, is_doctest=False
)
)
plain_block = None
non_code_directive = False
if match := doctest_code_start.match(line):
if plain_block:
buffer.extend(
_format_block(plain_block, exit_on_error, is_doctest=False)
)
plain_block = None
padding, code = match.groups()
doctest_block = [(line, line_no, padding, code)]
elif (
not no_plain
and not disable_format
and not last_line
and (match := plain_indent.match(line))
):
# print('start plain', line)
assert not doctest_block
# start of a plain block
padding, code = match.groups()
plain_block = [(line, line_no, padding, code)]
else:
buffer.append(line)
last_line = line
if doctest_block:
buffer.extend(
_format_block(doctest_block, exit_on_error, is_doctest=True)
)
if plain_block:
if non_code_directive:
buffer.extend(line for line, _, _, _ in plain_block)
else:
buffer.extend(
_format_block(plain_block, exit_on_error, is_doctest=False)
)
if buffer:
# if there is nothing in the buffer something strange happened so
# don't do anything
buffer.append("")
updated = "\n".join(buffer)
equal = original == updated
if not check:
print("..done. ", "No changes" if equal else "Changes detected")
if not equal:
# write only if there are changes to write
file.write_text(updated, "utf-8", newline="\n")
else:
if not check:
print(".. Nothing to write")
equal = bool(original) is False
if check:
if not equal:
print(f"File {file} would be formatted")
return equal
else:
return None
def iter_files(directory) -> Iterator[Path]:
yield from (home / directory).glob("./**/*.rst")
def main(
file: str | None,
directory: str,
exit_on_error: bool,
check: bool,
no_plain: bool,
):
if file is not None:
result = [format_file(Path(file), exit_on_error, check, no_plain)]
else:
result = [
format_file(doc, exit_on_error, check, no_plain)
for doc in iter_files(directory)
]
if check:
if all(result):
print("All files are correctly formatted")
exit(0)
else:
print("Some file would be reformated")
exit(1)
if __name__ == "__main__":
parser = ArgumentParser(
description="""Formats code inside docs using black. Supports \
doctest code blocks and also tries to format plain code block identifies as \
all indented blocks of at least 4 spaces, unless '--no-plain' is specified.
Plain code block may lead to false positive. To disable formatting on a \
file section the comment ``.. format: off`` disables formatting until \
``.. format: on`` is encountered or the file ends.
Another alterative is to use less than 4 spaces to indent the code block.
""",
formatter_class=RawDescriptionHelpFormatter,
)
parser.add_argument(
"-f", "--file", help="Format only this file instead of all docs"
)
parser.add_argument(
"-d",
"--directory",
help="Find documents in this directory and its sub dirs",
default="doc/build",
)
parser.add_argument(
"-c",
"--check",
help="Don't write the files back, just return the "
"status. Return code 0 means nothing would change. "
"Return code 1 means some files would be reformatted.",
action="store_true",
)
parser.add_argument(
"-e",
"--exit-on-error",
help="Exit in case of black format error instead of ignoring it. "
"This option is only valid for doctest code blocks",
action="store_true",
)
parser.add_argument(
"-l",
"--project-line-length",
help="Configure the line length to the project value instead "
"of using the black default of 88",
action="store_true",
)
parser.add_argument(
"-v",
"--verbose",
help="Increase verbosity",
action="store_true",
)
parser.add_argument(
"-n",
"--no-plain",
help="Disable plain code blocks formatting that's more difficult "
"to parse compared to doctest code blocks",
action="store_true",
)
args = parser.parse_args()
config = parse_pyproject_toml(home / "pyproject.toml")
BLACK_MODE = Mode(
target_versions=set(
TargetVersion[val.upper()]
for val in config.get("target_version", [])
),
line_length=config.get("line_length", DEFAULT_LINE_LENGTH)
if args.project_line_length
else DEFAULT_LINE_LENGTH,
)
VERBOSE = args.verbose
main(
args.file,
args.directory,
args.exit_on_error,
args.check,
args.no_plain,
)
|