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

Update clap to 4.0 #1124

Closed
wants to merge 2 commits into from
Closed

Update clap to 4.0 #1124

wants to merge 2 commits into from

Conversation

ljedrz
Copy link
Contributor

@ljedrz ljedrz commented Sep 30, 2022

Colored help output is currently not supported, but there are plans to work around it.

In addition, run cargo update as a drive-by.

Signed-off-by: ljedrz <ljedrz@gmail.com>
Signed-off-by: ljedrz <ljedrz@gmail.com>
@ljedrz
Copy link
Contributor Author

ljedrz commented Sep 30, 2022

CC #986.

@howardwu
Copy link
Member

howardwu commented Oct 5, 2022

@ljedrz how can we reenable the colored outputs for clap?

setting = clap::AppSettings::ColoredHelp

@raychu86
Copy link
Contributor

raychu86 commented Oct 5, 2022

Relevant info about Clap 4.* coloring - clap-rs/clap#4132 (comment)

@ljedrz
Copy link
Contributor Author

ljedrz commented Oct 5, 2022

Colors seem to have caused some issues and thus were removed for now. I've tried applying them using different clap means, but they didn't work.

@howardwu
Copy link
Member

howardwu commented Oct 5, 2022

Do we know if clap has a timeline for adding support for colors back in?

@ljedrz
Copy link
Contributor Author

ljedrz commented Oct 5, 2022

AFAIK not yet; however, during my search I stumbled upon an ANSI color feature, which we might be able to use to have colors sooner. I'll look into it.

@ljedrz
Copy link
Contributor Author

ljedrz commented Oct 7, 2022

I did take a look, and it doesn't seem doable with derive. I guess the update can wait to 4.x if we want to keep the help colors.

@howardwu
Copy link
Member

I'll close this PR for now and wait for 4.x to be released with colors. Thanks @ljedrz for being proactive about this.

@howardwu howardwu closed this Oct 18, 2022
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