HD-DEN-13801_SW Dev Sprint 79 MG

Activity

HD-DEN-13801-1 90

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 1h 14m 29 Fixed... but i dont recall making this change to this value
    Reviewer - Complete 1h 33m 52 RESOLVED in CODE WALKTHROUGH.
    Reviewer - Complete 0m 1 RESOLVED in CODE WALKTHROUGH
    wbracken  (deleted user)
    Reviewer completed
    Reviewer - Complete 45m 6 RESOLVED IN CODE WALKTHROUGH.
    Reviewer - 33% reviewed 14m 2 Last value is 0, should be 0.0. Do they need to be 0.0F?
    Total   3h 49m 90  
    #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.
    /HDDefs.h Changed
    Open in IDE #permalink
    /NVDataMgmtHDRecords.h Changed
    Open in IDE #permalink
    /firmware/App/Controllers/DGInterface.c Changed
    /firmware/App/Controllers/PresOccl.c Changed 3
    /firmware/App/Drivers/Battery.c Changed 3
    Open in IDE #permalink
    /firmware/App/Modes/ModePreTreat.c Changed 48
    /firmware/App/Modes/ModePreTreat.h Changed 6
    /firmware/App/Modes/ModeStandby.c Changed
    Open in IDE #permalink
    /firmware/App/Modes/Prime.c Changed 3
    /firmware/App/Modes/SelfTests.c Changed 27

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time