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

Flag to force or disable colored output #36

merged 2 commits into from Aug 1, 2019


Copy link

@orsinium orsinium commented Jul 31, 2019

  1. Disable colored output if --no-colors flag specified
  2. Force colored output if colors: true specified in the config.
  3. Disable colored output if colors: false specified in the config.
  4. Use old behavior by default: enable colors only for TTY
  5. IsTTY replaced by EnableColors, because now it's about colors, not TTY.

Close #33

Should I update some docs or do some tests? I haven't noticed a good place for it in the current codebase.

Thank you for the nice tool!

Copy link

@ai ai commented Jul 31, 2019

Should we take some checks from the JS library to detect color support?

Copy link

@Arkweid Arkweid commented Aug 1, 2019

Great work @orsinium! Thank you for contribution :)

@Arkweid Arkweid merged commit 1c35728 into evilmartians:master Aug 1, 2019
1 check passed
@orsinium orsinium deleted the colors-flag branch Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants