CesiumGS / obj2gltf

Convert OBJ assets to glTF
Apache License 2.0
1.71k stars 307 forks source link

Handle empty -i -o arguments in CLI #139

Closed lilleyse closed 6 years ago

lilleyse commented 6 years ago

Gives more helpful error messages if -i or -o are passed to the command line but are not followed by a filename.

Example: obj2gtlf -i file.obj -o will print Output path must be a filename Example: obj2gtlf -i will print Input path is required

I'd prefer if this check could be done from the yargs side. Does anyone know if yargs can demand a string option to be greater than zero length?

shehzan10 commented 6 years ago

Looks good. Thanks @lilleyse.