This is a list of all comments for DIALIN-PENDING-APPROAVAL-1.

Review Summary: No summary

General Comment by lbaloa on 21 January 2020, 09:53 
https://devapps.diality.us/cru/DIALIN-PENDING-APPROAVAL-1#c1159

  We need HD_TestScript.py to be move to a pytest type of
  function on the next revision of this software.

  Reply by Dara Navaei on 21 January 2020, 14:04
  > We will do it on the next revision

    Reply by Sean Nash on 22 January 2020, 09:31
    > RESOLVED.


----------------------------------------
File: Miscellaneous/HD_TestScript.py
Revision Comment by pmontazemi on 22 January 2020, 09:45 
https://devapps.diality.us/cru/DIALIN-PENDING-APPROAVAL-1#c1168

  Remove extra blank lines

  Reply by Dara Navaei on 22 January 2020, 09:58
  > Done

    Reply by pmontazemi on 22 January 2020, 10:09
    > RESOLVED.

----------------------------------------
File: DialityCoreCanProtocol.py
Revision Comment by lbaloa on 21 January 2020, 09:59 
https://devapps.diality.us/cru/DIALIN-PENDING-APPROAVAL-1#c1160

  change to \param and \return

  Reply by Dara Navaei on 21 January 2020, 14:04
  > Done

    Reply by Sean Nash on 22 January 2020, 09:45
    > RESOLVED.


---

ID: DIALIN-PENDING-APPROAVAL-1 https://devapps.diality.us/cru/DIALIN-PENDING-APPROAVAL-1

Title: DIALIN_Pending from DG code review
Statement of Objectives:


State: Closed

Summary: 

Author: Dara Navaei
Moderator: Dara Navaei
Reviewers: (2 active, 2 completed*)
     Sean Nash   (*)
     pmontazemi   (*)
     Behrouz NematiPour
     lbaloa