search

uber-node / tcurl

A command line utility to talk to tchannel servers
MIT License
36 stars 7 forks source link

Switch from Minimist to SHON #94

Closed kriskowal closed 8 years ago

Raynos commented 8 years ago

We should probably update manpage too.

kriskowal commented 8 years ago

I’ve upgraded this from WIP to the real deal™. Updated the man page (and the formatting errors in the man page). I’ve decided to dump the usage on -h since I doubt the flag summary is useful without any description. The flag summary is retained in the man page.

This is messy, but to make migration smooth, you can pass JSON with the flagged argument like --body {"nuck":{"language":"js","name":"tcurl"}} or use the positional argument for SHON like [ --nyuck [ --language js --name tcurl ] ].

Backward incompatibility is an error. Please note any you find.

Please review by running the branch anywhere you are using tcurl. Please note the errors for invalid usage. Please inform me of ergonomic issues.

In a future change, I intend to introduce -d raw-data or -d @file or -d- notation for as=raw, which will support streaming.

r @malandrew @Raynos @jcorbin @blampe @abhinav @breerly @ShanniLi

kriskowal commented 8 years ago

Verified health check in production with this branch.