LEAHI-SW-SCRIPTS-LDT-2727_Service - SW - 01 - Service - R&I

Activity

LEAHI-SW-SCRIPTS-LDT-2727-1 12

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 27m 5 added
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 0% reviewed      
    Reviewer - 100% reviewed 55m 7 This does not entirely follow the standard. But it is fin...
    Reviewer - 0% reviewed 0m    
    Reviewer - 0% reviewed      
    Total   1h 24m 12  
    #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

    Behrouz NematiPour

    All the script files need to follow the same format as the wifi_scan.sh. /sa ...

    All the script files need to follow the same format as the wifi_scan.sh.
    /sa this link

    Behrouz NematiPour

    Also, regarding the above format, each script needs to check whether a comman...

    Also, regarding the above format, each script needs to check whether a command fails and gracefully exit either way.

    Nicholas Ramirez

    added the checks and graceful exit

    added the checks and graceful exit

    Nicholas Ramirez

    Updated to follow format

    Updated to follow format

    /scripts/_errors_ Changed
    Open in IDE #permalink
    /scripts/bluetooth_paired_clear.sh Added 3
    Open in IDE #permalink
    /scripts/date_time_set.sh Added
    Open in IDE #permalink
    /scripts/decommission.sh Deleted
    Open in IDE #permalink
    /scripts/factory_reset.sh Added
    Open in IDE #permalink
    /scripts/rootsshaccess.sh Added 2
    Open in IDE #permalink
    /scripts/wifi_disconnect_network.sh Added 3
    Open in IDE #permalink
    /scripts/wifi_scan.sh Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time