[squash] remove deprecation for pipesCount

nodejs · starkwang · Jul 11, 2019 · Jul 12, 2019 · Jul 12, 2019 · 27ab4d9c0709c2f1d8cc6df243a4b0982b3953c1
commit 27ab4d9c0709c2f1d8cc6df243a4b0982b3953c1
diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md
@@ -2498,20 +2498,6 @@ Type: Runtime
 Passing a callback to [`worker.terminate()`][] is deprecated. Use the returned
 `Promise` instead, or a listener to the worker’s `'exit'` event.
 
-<a id="DEP0133"></a>
-### DEP0133: _readableState.pipesCount
-<!-- YAML
-changes:
-  - version: REPLACEME
-    pr-url: https://github.com/nodejs/node/pull/28583
-    description: Documentation-only.
--->
-
-Type: Documentation-only
-
-`_readableState.pipesCount` is deprecated. Please use
-`_readableState.pipes.length` instead.
-
 [`--http-parser=legacy`]: cli.html#cli_http_parser_library
 [`--pending-deprecation`]: cli.html#cli_pending_deprecation
 [`Buffer.allocUnsafeSlow(size)`]: buffer.html#buffer_class_method_buffer_allocunsafeslow_size

diff --git a/lib/_stream_readable.js b/lib/_stream_readable.js
@@ -30,7 +30,6 @@ const EE = require('events');
 const Stream = require('stream');
 const { Buffer } = require('buffer');
 
-const internalUtil = require('internal/util');
 const debug = require('internal/util/debuglog').debuglog('stream');
 const BufferList = require('internal/streams/buffer_list');
 const destroyImpl = require('internal/streams/destroy');
@@ -148,11 +147,11 @@ function ReadableState(options, stream, isDuplex) {
   }
 }
 
+// Legacy getter for `pipesCount`
 Object.defineProperty(ReadableState.prototype, 'pipesCount', {
-  get: internalUtil.deprecate(function() {
+  get() {
     return this.pipes.length;
-  }, '_readableState.pipesCount is deprecated. ' +
-  'Use _readableState.pipes.length instead.', 'DEP0133'),
+  }
 });
 
 function Readable(options) {

diff --git a/test/parallel/test-stream-pipescount-deprecation-warning.js b/test/parallel/test-stream-pipescount-deprecation-warning.js