Skip to content

♻️ refactor: using github.com/fatih/color for colors#9

Open
mhkarimi1383 wants to merge 2 commits intohelmwave:mainfrom
mhkarimi1383:feat-better-colors
Open

♻️ refactor: using github.com/fatih/color for colors#9
mhkarimi1383 wants to merge 2 commits intohelmwave:mainfrom
mhkarimi1383:feat-better-colors

Conversation

@mhkarimi1383
Copy link

@mhkarimi1383 mhkarimi1383 commented Apr 7, 2024

Hi

I have refactored the code to use https://github.com/fatih/color since it also has support for Windows (I'm Linux user, but in our team we have other users that are using Windows and we are using @halmwave itself and internal projects logging section is made by logrus sometimes), and making it easier to play with colors.

Also if needed I can move color.New from variable into the Format function, I don't know witch one will make it more clear to you.
and I change the code to manage enabling or disabling color using color package itself

Signed-off-by: Muhammed Hussein Karimi <info@karimi.dev>
Signed-off-by: Muhammed Hussein Karimi <info@karimi.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant