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

Document FORCE_COLOR=1 in kapp --help #632

Open
gberche-orange opened this issue Oct 26, 2022 · 2 comments
Open

Document FORCE_COLOR=1 in kapp --help #632

gberche-orange opened this issue Oct 26, 2022 · 2 comments
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request good first issue An issue that will be a good candidate for a new contributor snack This issue has been identified as lightweight and potentially quick to deliver.

Comments

@gberche-orange
Copy link

gberche-orange commented Oct 26, 2022

Describe the problem/challenge you have

In order to include a shortcut for kapp in k9s (see derailed/k9s#1830), I need to pipe kapp output to the less command.

Unless the export FORCE_COLOR=1 prereq, the kapp inspect | less -R fails to display colors. Same with --color or --tty flags

After searching a while, finally found the flag in https://kubernetes.slack.com/archives/CH8KCCKA5/p1600180101408100?thread_ts=1600180052.407500&cid=CH8KCCKA5

Describe the solution you'd like

kapp --help displays :

Environment Variables:
 - FORCE_COLOR: set to 1 to force colors to the printed. Useful in 'kapp ..| less -R' 

Anything else you would like to add:
[Additional information that will assist in solving the issue.]


Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@cppforlife
Copy link
Contributor

@renuy renuy added good first issue An issue that will be a good candidate for a new contributor carvel accepted This issue should be considered for future work and that the triage process has been completed snack This issue has been identified as lightweight and potentially quick to deliver. labels Nov 10, 2022
@renuy renuy moved this from To Triage to Unprioritized in Carvel Nov 10, 2022
@praveenrewar
Copy link
Member

Hey @gberche-orange! Thank you for creating this issue. The environment variable addition to kapp help section sounds good to me. Would you like to work on a PR for this?

@github-project-automation github-project-automation bot moved this to To Triage in Carvel Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
carvel accepted This issue should be considered for future work and that the triage process has been completed enhancement This issue is a feature request good first issue An issue that will be a good candidate for a new contributor snack This issue has been identified as lightweight and potentially quick to deliver.
Projects
Status: To Triage
Development

No branches or pull requests

4 participants