Done

 

From: John Lyle [mailto:notifications@github.com] 
Sent: 07 June 2013 10:44
To: webinos/webinos-pzp
Cc: Habib Virji
Subject: Re: [webinos-pzp] System test/ synchronization and remove friendlyName (#21)

 

In lib/pzp_connectPzh.js:

> @@ -19,11 +19,11 @@
>  /**
>   * Connects with PZH and handle respective events
>   */
> -var PzpOtherManager = require("./pzp_otherManager.js");
> +//var PzpOtherManager = require("./pzp_otherManager.js");

Again - I'd suggest killing this whole line, rather than commenting. Same is true for line 26. A very minor point.

—
Reply to this email directly or view it on GitHub <https://github.com/webinos/webinos-pzp/pull/21/files#r4585597> .  <https://github.com/notifications/beacon/XUg7godVN2O0jV1-5T0xO9NSB5W6rANo54spR6MQzO5RsvUzPYxpjHrlVPu0KcZl.gif>