fs: remove experimental warning for fs.promises

This has been warning for long enough, without any API changes in the last few months. PR-URL: #26581 Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Anto Aravinth <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]> Reviewed-By: James M Snell <[email protected]> Signed-off-by: Beth Griggs <[email protected]>
nodejs · Apr 9, 2019 · 3e386a7 · 3e386a7
1 parent ecac654
commit 3e386a7
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 9 deletions.
diff --git a/doc/api/fs.md b/doc/api/fs.md
@@ -3686,7 +3686,7 @@ this API: [`fs.write(fd, string...)`][].
 
 ## fs Promises API
 
-> Stability: 1 - Experimental
+> Stability: 2 - Stable
 
 The `fs.promises` API provides an alternative set of asynchronous file system
 methods that return `Promise` objects rather than using callbacks. The

diff --git a/lib/fs.js b/lib/fs.js
@@ -1894,13 +1894,10 @@ Object.defineProperties(fs, {
   },
   promises: {
     configurable: true,
-    enumerable: false,
+    enumerable: true,
     get() {
-      if (promises === null) {
+      if (promises === null)
         promises = require('internal/fs/promises');
-        process.emitWarning('The fs.promises API is experimental',
-                            'ExperimentalWarning');
-      }
       return promises;
     }
   }

diff --git a/test/parallel/test-fs-promises.js b/test/parallel/test-fs-promises.js
@@ -40,9 +40,8 @@ function nextdir() {
   return `test${++dirc}`;
 }
 
-// fs.promises should not be enumerable as long as it causes a warning to be
-// emitted.
-assert.strictEqual(Object.keys(fs).includes('promises'), false);
+// fs.promises should not enumerable.
+assert.strictEqual(Object.keys(fs).includes('promises'), true);
 
 {
   access(__filename, 'r')