Chore: Don't panic in KV command mods

[ashleymichal]

We panic a lot across the code base, but there's no reason to do it in these new modules. let's clean it up before we go to master.

Per #405 (comment)

[gabbifish]

Cleaned up our use of panic, as well as made our unwrap() calls in main.rs not risky (force .required(true) on args so Clap throws missing args before an unwrap() call for a missing arg could panic).

[ashleymichal]

where?

[gabbifish]

This was fixed in the commit linked to above (these were small enough changes I pushed directly to the feature branch; in the future, I can still go through PRs for small changes?).

[ashleymichal]

yah it just helps to track. since it's a feature branch merging isn't blocked so prs are still appreciated.

…

On Tue, Aug 27, 2019 at 10:40 AM Gabbi Fisher ***@***.***> wrote: This was fixed in the commit linked to above (these were small enough changes I pushed directly to the feature branch; in the future, I can still go through PRs for small changes?). — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#406>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABZA4P3QKJNPEWPLXLVLQ3TQGVDHBANCNFSM4IJYWUHQ> .

-- *Ashley Lewis* [email protected] <https://www.cloudflare.com/> 1 888 99 FLARE | www.cloudflare.com