tty: Remove deprecated setRawMode wrapper by geek · Pull Request #2528 · nodejs/node (original) (raw)

geek

@geek

@silverwind

👍.

Kind of related: we should also investigate deprecating tty.isatty in favor of ReadStream#isTTY, after which we can deprecate the whole module.

@silverwind silverwind added tty

Issues and PRs related to the tty subsystem.

semver-major

PRs that contain breaking changes and should be released in the next major version.

labels

Aug 24, 2015

@Fishrock123

@Fishrock123

@mscdex

@chrisdickinson

Adding this to the list of things to check.

@targos

@jasnell

@nodejs/ctc ... any further thoughts on this one?

@cjihrig

+1 to removal in v6

UPDATE: LGTM

@jasnell

Putting this on the ctc-agenda tomorrow for a quick sanity check. Don't believe it's controversial at all but worth a double check

@jasnell

geek added a commit that referenced this pull request

Nov 18, 2015

@geek @jasnell

PR-URL: #2528 Reviewed-By: Brian White mscdex@mscdex.net Reviewed-By: Colin Ihrig cjihrig@gmail.com Reviewed-By: James M Snell jasnell@gmail.com

@jasnell

@geek

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.

jasnell added a commit that referenced this pull request

Apr 26, 2016

@jasnell

The following significant (semver-major) changes have been made since the previous Node v5.0.0 release.