microsoft / vsts-cordova-tasks

Streamline CI setup for your Apache Cordova, PhoneGap, Ionic, or Cordova CLI compatible app using a set of useful pre-defined build steps for VS Team Services or TFS
http://go.microsoft.com/fwlink/?LinkID=691188
Other
25 stars 27 forks source link

fixing passing of env vars to properly not set null values #48

Closed ryuyu closed 8 years ago

ryuyu commented 8 years ago

Passing of environment variables from the exec.js to node-setup was setting all variables to a value, even if it was the string "null". This was causing subsequent checks of the environment to fail.

Update the exec logic to not set the values if null is returned from the task lib's getInput.

This should address a multitude of issues, including all the ones mentioned at https://github.com/Microsoft/vsts-cordova-tasks/issues/47

@Chuxel