Multi-Go icon indicating copy to clipboard operation
Multi-Go copied to clipboard

Suggestions

Open TheRedSpy15 opened this issue 5 years ago • 5 comments

Comment below on any suggestions. Anything from the text color, or even a new feature

TheRedSpy15 avatar Oct 08 '18 23:10 TheRedSpy15

While I’m working on the tests, do you want me to convert any TODOs I see in the comment into issues?

ScreamingTaco avatar Oct 16 '18 18:10 ScreamingTaco

Sure! that would be awesome if you could do that for me!

I've been meaning to do it, but there are quite a few & I kinda just got lazy

TheRedSpy15 avatar Oct 16 '18 19:10 TheRedSpy15

It seems a little excessive to panic on every error. I think it may be more reasonable to just write the error to stdout or stderr and then exit. What do you think?

gavinwade12 avatar Oct 26 '18 22:10 gavinwade12

Also, there are already quite a few tasks, and I see you have TODOs for adding even more. Perhaps the project should be migrated over to something like https://github.com/spf13/cobra, which should offer better structure and take some of the work out of our hands. It also has lots of other cool features like autocomplete and intelligent suggestions.

gavinwade12 avatar Oct 26 '18 22:10 gavinwade12

@gavinwade12 I prefer using panic, as that is the primary use of panic (as far as I know). However, I'm always open to just writing the error and exiting.

Also, I've been reading up on cobra. Unfortunately, I'm still not entirely certain as to what it's meant for or how to incorporate it. But I'll defiantly being looking into it. I'll never turn down something that'll make developing easier!

TheRedSpy15 avatar Oct 27 '18 02:10 TheRedSpy15