LEAHI-DD-FIRMWARE-LDT-4734_Water Treatment and Degassing - FW Iteration - 5/4:

Activity

LEAHI-DD-FIRMWARE-LDT-4734-1 18

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 10m 5 fixed. thanks!
    Reviewer - Complete 7m 3 Add blank line (should be 2 before test support banner).
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - Complete 18m 6 space after if.
    Reviewer - 0% reviewed 2m 1 so, are we removing this? I think it is better to have th...
    Total   38m 18  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    There are no general comments on this review.
    /firmware/.../Drivers/ConductivitySensors.c Changed 10
    Open in IDE #permalink
    /firmware/.../Drivers/ConductivitySensors.h Changed
    Open in IDE #permalink
    /firmware/App/Monitors/Conductivity.c Changed 6
    Open in IDE #permalink
    /firmware/App/Monitors/Conductivity.h Changed 2
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time