feat: remove owner related code as well as stale polyfills (#53)

BREAKING CHANGE: this removes the `owner` option from all methods that previously supported it, as well as the `withOwner` and `withOwnerSync` methods
npm · Oct 11, 2022 · e666309 · e666309
1 parent 7ec9b2d
commit e666309
Show file tree

Hide file tree

Showing 25 changed files with 7 additions and 1,927 deletions.
diff --git a/README.md b/README.md
@@ -5,28 +5,8 @@ polyfills, and extensions, of the core `fs` module.
 ## Features
 
 - all exposed functions return promises
-- `fs.rm` polyfill for node versions < 14.14.0
-- `fs.mkdir` polyfill adding support for the `recursive` and `force` options in node versions < 10.12.0
-- `fs.copyFile` extended to accept an `owner` option
-- `fs.mkdir` extended to accept an `owner` option
-- `fs.mkdtemp` extended to accept an `owner` option
-- `fs.writeFile` extended to accept an `owner` option
-- `fs.withTempDir` added
-- `fs.withOwner` added
-- `fs.withOwnerSync` added
 - `fs.cp` polyfill for node < 16.7.0
-
-## The `owner` option
-
-The `copyFile`, `mkdir`, `mkdtemp`, `writeFile`, and `withTempDir` functions
-all accept a new `owner` property in their options. It can be used in two ways:
-
-- `{ owner: { uid: 100, gid: 100 } }` - set the `uid` and `gid` explicitly
-- `{ owner: 100 }` - use one value, will set both `uid` and `gid` the same
-
-The special string `'inherit'` may be passed instead of a number, which will
-cause this module to automatically determine the correct `uid` and/or `gid`
-from the nearest existing parent directory of the target.
+- `fs.withTempDir` added
 
 ## `fs.withTempDir(root, fn, options) -> Promise`
 

diff --git a/lib/common/owner-sync.js b/lib/common/owner-sync.js
diff --git a/lib/common/owner.js b/lib/common/owner.js
diff --git a/lib/copy-file.js b/lib/copy-file.js
diff --git a/lib/errors.js → lib/cp/errors.js b/lib/errors.js → lib/cp/errors.js
diff --git a/lib/cp/polyfill.js b/lib/cp/polyfill.js
@@ -23,7 +23,7 @@ const {
   ERR_FS_CP_UNKNOWN,
   ERR_FS_EISDIR,
   ERR_INVALID_ARG_TYPE,
-} = require('../errors.js')
+} = require('./errors.js')
 const {
   constants: {
     errno: {

diff --git a/lib/index.js b/lib/index.js
@@ -1,12 +1,5 @@
 module.exports = {
   ...require('./fs.js'),
-  copyFile: require('./copy-file.js'),
   cp: require('./cp/index.js'),
-  mkdir: require('./mkdir.js'),
-  mkdtemp: require('./mkdtemp.js'),
-  rm: require('./rm/index.js'),
   withTempDir: require('./with-temp-dir.js'),
-  withOwner: require('./with-owner.js'),
-  withOwnerSync: require('./with-owner-sync.js'),
-  writeFile: require('./write-file.js'),
 }
diff --git a/lib/mkdir.js b/lib/mkdir.js
diff --git a/lib/mkdtemp.js b/lib/mkdtemp.js
diff --git a/lib/rm/index.js b/lib/rm/index.js