Skip to content

Commit 03088a9

Browse files
albuzenetAlex
and
Alex
authored
BUG validate_docstrings.py flake8 PermissionError (#51244)
remove context manager Co-authored-by: Alex <[email protected]>
1 parent b37321c commit 03088a9

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

Diff for: scripts/validate_docstrings.py

+6-1
Original file line numberDiff line numberDiff line change
@@ -207,7 +207,9 @@ def validate_pep8(self):
207207
)
208208

209209
error_messages = []
210-
with tempfile.NamedTemporaryFile(mode="w", encoding="utf-8") as file:
210+
211+
file = tempfile.NamedTemporaryFile(mode="w", encoding="utf-8", delete=False)
212+
try:
211213
file.write(content)
212214
file.flush()
213215
cmd = ["python", "-m", "flake8", "--quiet", "--statistics", file.name]
@@ -217,6 +219,9 @@ def validate_pep8(self):
217219
messages = stdout.strip("\n")
218220
if messages:
219221
error_messages.append(messages)
222+
finally:
223+
file.close()
224+
os.unlink(file.name)
220225

221226
for error_message in error_messages:
222227
error_count, error_code, message = error_message.split(maxsplit=2)

0 commit comments

Comments
 (0)