onepub-dev / dcli

An extensive library and tooling for building console/cli applications and scripts using the Dart programming language.
246 stars 27 forks source link

setEnv should not allow nulls. #62

Closed bsutton closed 4 years ago

bsutton commented 4 years ago

call Runproces.start will result in an exception is an environment variable has been set to null.

As such we should either set the value to an empty string or clear the variable. I'm inclined to clear the variable out.

bsutton commented 4 years ago

setEnv now removes an environment var if a null value is passed.