Skip to content

Commit 086fffd

Browse files
fix: rename unused _out to _, add explicit workflow permissions
Agent-Logs-Url: https://github.com/codeSamuraii/pyfuse/sessions/925ac194-398a-43f8-aba9-4fc0abfedaad Co-authored-by: codeSamuraii <17270548+codeSamuraii@users.noreply.github.com>
1 parent bd61268 commit 086fffd

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

.github/workflows/integration.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,10 @@ concurrency:
1010
group: ${{ github.workflow }}-${{ github.ref }}
1111
cancel-in-progress: true
1212

13+
# Restrict the workflow token to the minimum required permissions.
14+
permissions:
15+
contents: read
16+
1317
env:
1418
PYTHON_VERSION: "3.12"
1519

tests/integration/conftest.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ def worker_process(
120120
port = parsed.port or 9748
121121
if not _wait_for_tcp(host, port, timeout=20.0):
122122
proc.terminate()
123-
_out, err = proc.communicate(timeout=5)
123+
_, err = proc.communicate(timeout=5)
124124
raise RuntimeError(
125125
f"Worker broker did not start on {host}:{port}.\n"
126126
f"stderr:\n{err.decode()}"
@@ -131,7 +131,7 @@ def worker_process(
131131
time.sleep(4)
132132

133133
if proc.poll() is not None:
134-
_out, err = proc.communicate()
134+
_, err = proc.communicate()
135135
raise RuntimeError(
136136
f"Worker process exited prematurely (rc={proc.returncode}).\n"
137137
f"stderr:\n{err.decode()}"

0 commit comments

Comments
 (0)