Skip to content

Commit 8ad580e

Browse files
committed
Remove undocumented stderr parameter
subprocess.check_call(args, stderr=True) merges child stderr into parent stdout instead of separating streams. CPython accepts it because bool inherits from int, interpreting True as file descriptor 1 (stdout). This redirects child stderr to parent stdout, mixing diagnostic output with normal output. This removes stderr parameter to use default (stderr=None), which inherits parent stderr per subprocess documentation, properly separating stdout and stderr streams.
1 parent 67819bf commit 8ad580e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

runtest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ def _notebook_run(path):
2222
"--allow-errors",
2323
"--ExecutePreprocessor.timeout=60",
2424
"--output", fout.name, path]
25-
subprocess.check_call(args, stderr=True)
25+
subprocess.check_call(args)
2626

2727
fout.seek(0)
2828
nb = nbformat.read(fout, nbformat.current_nbformat)

0 commit comments

Comments
 (0)