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

Add the possibility to disable color #555

Merged
merged 2 commits into from
Jul 3, 2018
Merged

Conversation

shamsak58
Copy link
Contributor

No Color users add support

No Color users add support
@@ -14,6 +14,8 @@ case class CliOptions(
version: Boolean = false,
@HelpMessage("If set, print out debugging information to stderr.")
verbose: Boolean = false,
@HelpMessage("If set, do not color output.")
noColor: Boolean = false,
Copy link
Contributor

Choose a reason for hiding this comment

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

Would it be better to respect the environment variable NO_COLOR? http://no-color.org/

Copy link
Contributor

Choose a reason for hiding this comment

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

Indeed, this would be a great default. I'll commit a message adding it so that we can unblock this pull request next week.

@jvican jvican changed the title add no-color Add the possibility to disable color Jun 21, 2018
@jvican jvican added enhancement cli ergonomics Any change that affects developer ergonomics and the easiness of use of bloop. community Any change or proposal that is contributed by the Open Source Community. labels Jun 29, 2018
@jvican
Copy link
Contributor

jvican commented Jul 3, 2018

Added bloop to the list of software that now supports NO_COLOR jcs/no_color#30, thanks for the suggestion @olafurpg

@jvican
Copy link
Contributor

jvican commented Jul 3, 2018

Great job @shamsak58 🎉 Thank you for this pull request.

@jvican jvican merged commit de0cdd4 into scalacenter:master Jul 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli community Any change or proposal that is contributed by the Open Source Community. enhancement ergonomics Any change that affects developer ergonomics and the easiness of use of bloop.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants