这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Jul 30, 2019

Added a --no-color flag to address: (fix #2072 )

Makes it so that outputs wont get colorized

Affected components

  • CLI

Solution and Design

Used logrus' DisableColor

Steps to test and verify

  1. run make build
  2. run the $PATH_TO_OUTPUT/cli-hasura-{{.OS}}-{{.Arch}} version to produce the colored output
  3. run the $PATH_TO_OUTPUT/cli-hasura-{{.OS}}-{{.Arch}} version --no-color to produce the non colored output

@ghost ghost requested a review from shahidhk as a code owner July 30, 2019 05:36
@CLAassistant
Copy link

CLAassistant commented Jul 30, 2019

CLA assistant check
All committers have signed the CLA.

@netlify
Copy link

netlify bot commented Jul 30, 2019

Deploy preview for hasura-docs ready!

Built with commit a82b2ae

https://deploy-preview-2634--hasura-docs.netlify.com

@hasura-bot
Copy link
Contributor

Review app for commit 592593a deployed to Heroku: https://hge-ci-pull-2634.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2634-592593a1

@ecthiender ecthiender added the c/cli Related to CLI label Jul 30, 2019
@hasura-bot
Copy link
Contributor

Review app for commit a82b2ae deployed to Heroku: https://hge-ci-pull-2634.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2634-a82b2ae7

Copy link
Member

@shahidhk shahidhk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot @dooven! LGTM.

@shahidhk shahidhk changed the title Add flag to disable CI colors add flag to disable ci colors (close #2072) Jul 30, 2019
@shahidhk shahidhk merged commit 7f06470 into hasura:master Jul 30, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-2634.herokuapp.com is deleted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/cli Related to CLI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add flag to disable term color when runnning on CI

4 participants