node: deprecate process.EventEmitter#5049
Conversation
|
/cc @ChALkeR Does anyone use this anywhere? |
|
FWIW here is a github search for it as well. https://github.com/search?p=2&q=process.EventEmitter&type=Code&utf8=%E2%9C%93 |
|
LGTM |
1 similar comment
|
LGTM |
|
CI: https://ci.nodejs.org/job/node-test-pull-request/1524/ Was the decision that we could deprecate in a minor version, or did we go with major? |
|
@mscdex This is used, though the usage is not high. Will post a list a bit later today. |
|
LGTM |
1 similar comment
|
LGTM |
|
@evanlucas you can get better results by quoting. |
|
oh nice |
|
https://gist.github.com/ChALkeR/232fdf64be9c34c9c733, sorted by downloads count per month. I'm +1 for deprecating this, but in a |
|
Major works for me |
|
LGTM, looks to be low-impact. |
|
CI https://ci.nodejs.org/job/node-test-pull-request/1550/ one more time before landing since the last one was red. |
The comment stating it was deprecated was added in 2011 via 4ef8f06. It is time to actually deprecate it. PR-URL: nodejs#5049 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
|
Landed in 25751be. Thanks |
The comment stating it was deprecated was added in 2011 via 4ef8f06. It is time to actually deprecate it. PR-URL: nodejs#5049 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com> Reviewed-By: Roman Reiss <me@silverwind.io>
The comment stating it was deprecated was added in 2011 via
4ef8f06. It is time to actually deprecate it.