Lines of code count not available
Added but crucible is not showing this change. Please check bitbucket.
These variables are correctly initialized to the first enum in the list.
Not needed because the retry mechanism is going to be deleted in the next ticket.
I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.
What is happening here? Not clear if we need cartridge param. Let's remove it for now.
Remove extra blank line.
Comment these out again.
Move this to right before syringe pump controller below. Add comment "Execute pinch valve command state machine."
Move this to right after valves controller exec below.
Restore this.
uncomment this line.
Remove test code.
I think we should have handle functions that return next state for each of these cases (just like mode state machines).
Remove extra blank lines and comment.
Remove extra blank lines.
Remove this comment?
Just have a comment here that says "Ok - do nothing."
If we don't need this code, remove it.
Add a TODO comment to restore this when Noe fixes FPGA
Remove blank line.
Why commented out?
Missing /// comment.
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.