Skip to content

Commit

Permalink
fix(api): Do not pass loop arg to subproc
Browse files Browse the repository at this point in the history
asyncio subprocess used to need loop; now it doesn't. For a
minimum-affected fix, just don't pass loop to the subprocess invocations
while keeping it in the top level environment.

Closes RQA-2417
  • Loading branch information
sfoster1 committed Feb 29, 2024
1 parent d792317 commit da579f4
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion api/src/opentrons/hardware_control/modules/update.py
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ async def update_firmware(
kwargs: Dict[str, Any] = {
"stdout": asyncio.subprocess.PIPE,
"stderr": asyncio.subprocess.PIPE,
"loop": loop,
}
successful, res = await module.bootloader()(
flash_port_or_dfu_serial, str(firmware_file), kwargs
Expand Down

0 comments on commit da579f4

Please sign in to comment.