From 7bd48896e912c7d5cfd12b6e6c314d13d91e74c5 Mon Sep 17 00:00:00 2001 From: James M Snell Date: Mon, 14 May 2018 17:55:46 -0700 Subject: [PATCH] fs: move SyncWriteStream to end-of-life This was deprecated in 8.x or 9.x. It was never intended for public use. PR-URL: https://github.com/nodejs/node/pull/20735 Reviewed-By: Rich Trott Reviewed-By: Colin Ihrig Reviewed-By: Daniel Bevenius Reviewed-By: Yuta Hiroto Reviewed-By: Jackson Tian Reviewed-By: Ruben Bridgewater Reviewed-By: Trivikram Kamat --- doc/api/deprecations.md | 5 +++-- lib/fs.js | 11 ----------- 2 files changed, 3 insertions(+), 13 deletions(-) diff --git a/doc/api/deprecations.md b/doc/api/deprecations.md index 4b922045556..fb367553a5e 100644 --- a/doc/api/deprecations.md +++ b/doc/api/deprecations.md @@ -528,10 +528,11 @@ The [`util._extend()`][] API has been deprecated. ### DEP0061: fs.SyncWriteStream -Type: Runtime +Type: End-of-Life The `fs.SyncWriteStream` class was never intended to be a publicly accessible -API. No alternative API is available. Please use a userland alternative. +API and has been removed. No alternative API is available. Please use a userland +alternative. ### DEP0062: node --debug diff --git a/lib/fs.js b/lib/fs.js index dee437c5966..6fc245758fa 100644 --- a/lib/fs.js +++ b/lib/fs.js @@ -2360,14 +2360,3 @@ WriteStream.prototype.close = function(cb) { // There is no shutdown() for files. WriteStream.prototype.destroySoon = WriteStream.prototype.end; - -// SyncWriteStream is internal. DO NOT USE. -// This undocumented API was never intended to be made public. -var SyncWriteStream = internalFS.SyncWriteStream; -Object.defineProperty(fs, 'SyncWriteStream', { - configurable: true, - get: internalUtil.deprecate(() => SyncWriteStream, - 'fs.SyncWriteStream is deprecated.', 'DEP0061'), - set: internalUtil.deprecate((val) => { SyncWriteStream = val; }, - 'fs.SyncWriteStream is deprecated.', 'DEP0061') -});