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

Do not output colorized config when environment does not support it #270

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cfunkhouser
Copy link

@cfunkhouser cfunkhouser commented Jan 21, 2025

Currently, regardless of the COLOR / NOCOLOR environment settings, the output of restish api show $API is output in color. This is quite inconvenient when piping the output to other programs which are unable to process the ANSI color codes. In my case, this specifically breaks piping the output into jq.

For example, with the current release:

$ restish api show default | jq '.profiles.default.headers | keys[]'
jq: parse error: Invalid numeric literal at line 1, column 2

With this fix included:

$ restish api show mc | jq '.profiles.default.headers | keys[]'
"authorization"

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.

1 participant