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

rollup + rustfmt + switch to GitHub Actions + support NO_COLOR #24

Merged
merged 12 commits into from
Jan 11, 2020

Conversation

BurntSushi
Copy link
Owner

No description provided.

mookid and others added 11 commits January 11, 2020 07:59
It looks like `intense` wasn't previously being considered in `clear`
and `is_none`. This commit fixes that and adds some regression tests.
The default remains unchanged: whenever `set_color` is called, the
underlying terminal is reset when using ANSI escape sequences. The new
`reset` option permits callers to disable this, and thus managing
resetting themselves.

Fixes #17, Closes #18
This also deprecates the wincolor crate. Its implementation was folded
verbatim into the winapi-util crate.
This adds support for respecting the NO_COLOR environment variable,
whose spec is defined here: https://no-color.org/

The relevant portion is:

    > All command-line software which outputs text with ANSI color
    > added should check for the presence of a NO_COLOR environment
    > variable that, when present (regardless of its value), prevents
    > the addition of ANSI color.

As such, termcolor will now detect the presence of the NO_COLOR
environment variable. If it exists, and the color choice is "auto," then
colors will be suppressed.

Closes #1186
@BurntSushi BurntSushi changed the title rollup + rustfmt + switch to GitHub Actions rollup + rustfmt + switch to GitHub Actions + support NO_COLOR Jan 11, 2020
@BurntSushi BurntSushi merged commit 210869b into master Jan 11, 2020
@BurntSushi BurntSushi deleted the ag/updates branch January 11, 2020 15:01
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.

4 participants