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

PR for String parsing for TrueColor (closest match), and TrueColor for Hex. #118

Open
EvanCarroll opened this issue Sep 22, 2022 · 4 comments

Comments

@EvanCarroll
Copy link

EvanCarroll commented Sep 22, 2022

It seems at $work, we use a fork of a fork of this project. Reviewing the code, I want to know if it's possible to get it included. It seems the original adds the ability to fall back if TrueColor isn't available. Our fork we use at work adds to that the ability to parse the colors from a string. This is all hidden behind a feature flag, so no added deps if not wanted. There are also tests.

Is this something the project would be interested in including?

The branch we use at work is https://github.com/jtk18/colored/tree/parse_truecolor
This is forked from https://github.com/spenserblack/colored

It seems spenserblack has an open PR up here. #90 But that PR has no feedback. Can more work be done to get that included upstream? If so, what can I do to help?

@EvanCarroll EvanCarroll changed the title String parsing for TrueColor. PR for String parsing for TrueColor (closest match), and TrueColor for Hex. Sep 22, 2022
@hwittenborn
Copy link
Member

Hey! I recently joined the team for this project, and I'd like to start getting a lot of these issues and PRs cleaned up. I'll give that PR a look soon so it can get merged in :)

@mackwic
Copy link
Collaborator

mackwic commented Jul 2, 2023

Hi, old maintainer here with not enough time to review and understand everything. Notably, this PR adds a new control environment ($COLORTERM) which I did not knew, and would like to understand better to write some documentation for the end users. The code seems good though.

@EvanCarroll do you think that $work would be open for some sponsorship or this project, which would help us prioritize work on colored?

@EvanCarroll
Copy link
Author

I don't think so, re sponsorship. Perhaps you should up to other maintainers that have more time. Seems @spenserblack may want to step up to the plate.

@spenserblack
Copy link
Collaborator

spenserblack commented Jul 5, 2023

Actually, all my PRs here are years old, and I haven't used Rust in a while, so I'm not sure if I'm the best candidate.

But I'd be willing to at least help out with answering simple issues and reviewing some PRs. So maybe not "up to the plate," but "plate adjacent" 😆

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

No branches or pull requests

4 participants