Skip to content

Commit

Permalink
Update todo comment to be more specific.
Browse files Browse the repository at this point in the history
  • Loading branch information
SyntaxColoring committed Apr 8, 2024
1 parent a880096 commit 50e1706
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions api/src/opentrons/protocol_engine/protocol_engine.py
Original file line number Diff line number Diff line change
Expand Up @@ -304,8 +304,9 @@ def estop(
running_or_next_queued_id = (
self._state_store.commands.get_running_command_id()
or self._state_store.commands.get_queue_ids().head(None)
# TODO(mm, 2024-04-02): Is it possible for the next queued command to
# be a setup command? That wouldn't show up in commands.get_queue_ids().
# TODO(mm, 2024-04-02): This logic looks wrong whenever the next queued
# command is a setup command, which is the normal case in maintenance
# runs. Setup commands won't show up in commands.get_queue_ids().
)
running_or_next_queued = (
self._state_store.commands.get(running_or_next_queued_id)
Expand Down

0 comments on commit 50e1706

Please sign in to comment.