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

Force colors in Windows when running from electron #1127

Closed
brian-mann opened this issue Dec 24, 2017 · 2 comments
Closed

Force colors in Windows when running from electron #1127

brian-mann opened this issue Dec 24, 2017 · 2 comments
Assignees
Labels
type: enhancement Requested enhancement of existing feature
Milestone

Comments

@brian-mann
Copy link
Member

Per this issue in electron: electron/electron#11488

When running electron in Windows, isTTY ends up being false. To preserve the native behavior of Windows, we're opting not to monkey patch this. However, we will force colors to be output correctly (if originally supported) thus making Windows look nice™.

@brian-mann brian-mann added the type: enhancement Requested enhancement of existing feature label Dec 24, 2017
@brian-mann brian-mann self-assigned this Dec 24, 2017
@brian-mann brian-mann added this to the 1.4.1 milestone Dec 24, 2017
brian-mann added a commit that referenced this issue Dec 25, 2017
…r when xvfb exits with status code 1, force tty in linux, handle colors in windows, enable logging cypress:xvfb stderr

* cli: fixes #838 start cypress in dev by routing through the CLI

- matches how we run in production better to keep parity and consistency

* cli: add coerceFalse for clarity

* cli: add global flag, update to work with windows

* server: bring into parity with root scripts

* cli: just execute start script directly to work with windows

* cli: if colors are supported then force them via env vars

- this fixes windows not displaying colors from electron because by
default isTTY is false (due to electron)

* cli: fixes #921 don't ignore stderr, inherit stdio on everything except when linux + xvfb

- filter out stderr messages coming from Xlib or libudev (from xvfb)

* cli, server: force stderr tty so that normalize tty behavior when piping

* server: drop in supports color so debug outputs more colors!

* server: remove empty line

* root: refer to cypress not monorepo

* cli: make util.supportsColor return boolean

* cl: add tests around spawn behavior with forcing colors, tty, and stdio configuration

* cli: handle xvfb onStderrData callback to output debug information

* cli: handle non zero exit code error from xvfb with special message
@brian-mann
Copy link
Member Author

Fixed by #1120.

@brian-mann
Copy link
Member Author

Released in 1.4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Requested enhancement of existing feature
Projects
None yet
Development

No branches or pull requests

1 participant