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

Skip colourisation for dumb terminals #56

Merged
merged 1 commit into from
Jul 29, 2015
Merged

Skip colourisation for dumb terminals #56

merged 1 commit into from
Jul 29, 2015

Conversation

andreastt
Copy link
Contributor

Dumb terminals can only interpret a limited number of control codes, and colour escape codes will be written as plain text (garbage) on these terminals.

This can be tested with making sure the following program only outputs “ERROR: dumb”:

node -e 'require("./cli.js").error("dumb")'

Dumb terminals can only interpret a limited number of control codes,
and colour codes will be written as plain text (garbage) on these
terminals.

This can be tested with making sure the following program only
outputs “ERROR: dumb”:

	node -e 'require("./cli.js").error("dumb")'
@andreastt
Copy link
Contributor Author

@chriso Any chance you could look at this?

@chriso
Copy link
Contributor

chriso commented Jul 29, 2015

Sorry for the epic delay.

chriso added a commit that referenced this pull request Jul 29, 2015
Skip colourisation for dumb terminals
@chriso chriso merged commit ae26267 into node-js-libs:master Jul 29, 2015
@chriso
Copy link
Contributor

chriso commented Jul 29, 2015

Available in 0.9.0.

@andreastt
Copy link
Contributor Author

Thank you very much! This is (eventually) going to make using Node.js a whole lot better for me.

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.

2 participants