lein-zprint icon indicating copy to clipboard operation
lein-zprint copied to clipboard

Avoid NPE if run without args

Open amosjyng opened this issue 3 years ago • 2 comments

Like probably many others, I didn't bother reading the documentation before running lein zprint :)

The NPE made me think the tool was broken at first. I think printing the help message instead will make it more obvious that it has to be run with actual arguments

amosjyng avatar Oct 10 '21 19:10 amosjyng

Thanks for doing this! Good point about the NPE with no arguments. For what it is worth, I've found that the relatively recent changes to the pre-built binaries to add the -w switch and its various friends have shifted my usage of zprint from using lean zprint to using the binaries directly. Largely because they are so much faster to run. Of course, if you can't use them, then lein-zprint is fine too.

kkinnear avatar Oct 10 '21 22:10 kkinnear

Ah interesting, I hadn't looked into zprint binaries. Though what I like about using a lein plugin is that lein downloads everything for me :)

amosjyng avatar Oct 10 '21 23:10 amosjyng