Skip to content
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

Export Color type #336

Closed

Conversation

natealcedo
Copy link

The PR exposes a type for colors supported by chalk. The use case of this is so that consumers of this library can programatically check if a color is available.

One example is in this PR. jestjs/jest#8025

I am hardcoding the possible values that a color can be. https://github.com/facebook/jest/pull/8025/files#diff-6170a10d44dcb3222b64fba6427d69c7R234

@sindresorhus
Copy link
Member

Makes sense, but I think we should split it in ForegroundColor, BackgroundColor, and Color, which is a combination of the two. Maybe we should also expose the modifiers too?

@SimenB
Copy link
Contributor

SimenB commented Jun 25, 2019

@natealcedo ping 🙂

@natealcedo
Copy link
Author

Hello again! Sorry I neglected this. Thanks @SimenB for reminding me. Let me take a look at this again over the weekend :)

@natealcedo
Copy link
Author

Hi actually, I've been terribly busy with work and my personal life. I'm not sure if I can pick this up anytime soon if anyone wants to take a stab at this. :)

@SimenB
Copy link
Contributor

SimenB commented Jul 15, 2019

Opened up #357

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants