-
Notifications
You must be signed in to change notification settings - Fork 619
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add RGB API support #225
Add RGB API support #225
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great thanks 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple nitpicks on the comments but, LGTM 🚀
API seems straightforward and easy to use. It's a great addition to the existing supported ANSI color palette.
Thanks for picking this up 👍🏼
Thanks for to the reviews. I want to keep this open for some time to try it out before I merge it. |
Another vote of confidence. Worked great |
Hey @fatih, did you have a chance to take a look at this PR? |
@bschaatsbergen yeap, I'll merge this soon. Hadn't time go over it. |
Signed-off-by: Fatih Arslan <ftharsln@gmail.com>
Co-authored-by: Bruno Schaatsbergen <git@bschaatsbergen.com>
This change will be great! Any chance we can get it released soon-ish? |
@amterp thanks for the ping, I'll cut a release today. |
Here is the release: https://github.com/fatih/color/releases/tag/v1.18.0 |
Looks good! 😄 Thanks @fatih |
This PR adds support for a new RGB API to allow to print 24-bit colors. Opening for feedback, and will probably merge it as it is. If anyone wants to try it out, check the
TestRGB
test case or the newly added README.md docs.