Closed benmcginnis closed 7 years ago
Hi @benmcginnis, thanks a lot for your contribution, I just merged your PR.
There is only a small link I am going to change: I will make the flags -json and -csv a string option, so a filename can be passed where the exported data should be saved.
Hi @rverton you're welcome, and great! I thought about doing that but it seemed like you're nifty webpage view on the data relied on the json output being "hosts.json"
I just pushed an update where it is possible to specify an filename. I hope this feels more "natural" when working with the export function.
@rverton great implementation! I've added a CSV export as well as made a few improvements to fix some gometalinter warnings and to remove the pointer usage on the flags for the easer
flag.BoolVar
style. Let me know what you think!