junyu-w / corgi

Corgi is a command-line workflow manager that helps with your repetitive command usages by organizing them into reusable snippet

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Both fatih/color and defined consts for shell colors?

andres-lowrie opened this issue · comments

Hello

Was curious as to why you're using both the fatih/color package and the escape characters defined in the util package

https://github.com/DrakeW/corgi/blob/develop/util/util.go#L23

For example in this file they're both being used:

const values here
https://github.com/DrakeW/corgi/blob/develop/snippet/step.go#L62

and the color package here
https://github.com/DrakeW/corgi/blob/develop/snippet/step.go#L96

I can create a Pull request consolidating the usage to the fatih/color package but before doing so I wanted to check to see if there was something I'm missing in terms of a need for both of them

hi @andres-lowrie , the escape chars are only used for exporting to shell script, if you can make them consistent by using the color package and keep output color printed when executing the exported shell script, please go ahead and open a PR!