LEAHI-DD-FIRMWARE-LDT-2030_Blood Leak - DD

Activity

LEAHI-DD-FIRMWARE-LDT-2030-2 39

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 2h 33m 16 I think you are seeing previous commit. It is not there i...
    Reviewer - 0% reviewed 5m 1 Vendor - TEL - Sameer Poyil this structure is not used to...
    Reviewer - 0% reviewed      
    Reviewer - 35% reviewed 11m    
    Reviewer - 0% reviewed      
    Reviewer - 100% reviewed 31m 8 Dara Navaei, this is not a CAN message payload, but it do...
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed 4m    
    Reviewer - 0% reviewed      
    Reviewer - 100% reviewed 3h 9m 14 is it merge issue ?
    Total   6h 36m 39  
    #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/App/Drivers/BloodLeakDriver.c Added 12
    Open in IDE #permalink
    /firmware/App/Drivers/BloodLeakDriver.h Added 8
    Open in IDE #permalink
    /firmware/App/Monitors/BloodLeak.c Changed 13
    Open in IDE #permalink
    /firmware/App/Monitors/BloodLeak.h Changed 6
    Open in IDE #permalink
    /firmware/App/Services/Messaging.c Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time