Skip to content

fs: add recursive option to rmdir()

Rodrigo Muino Tomonari requested to merge github/fork/cjihrig/rimraf into master

This PR adds a recursive option to fs.rmdir(), fs.rmdirSync(), and fs.promises.rmdir(). The implementation is a port of the npm module rimraf. I added an option to rmdir() to match the approach taken with core's recursive mkdir().

This is my alternative to https://github.com/nodejs/node/pull/28208 from https://github.com/nodejs/node/pull/28208#pullrequestreview-275046863.

I'm marking this PR as a work in progress, as I haven't added docs yet.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports