-
-
Notifications
You must be signed in to change notification settings - Fork 3k
Closed
Labels
area: repository toolingconcerning ease of contributionconcerning ease of contributionstatus: accepting prsMocha can use your help with this one!Mocha can use your help with this one!
Description
Tooling Suggestion Checklist
- I have tried restarting my IDE and the issue persists.
- I have pulled the latest
main
branch of the repository. - I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
- I have searched for related issues and issues with the
faq
label, but none matched my issue. - I want to provide a PR to resolve this
Overview
I ran eslint-plugin-depend
on the Mocha repo. It had two reports about fs-extra
:
/Users/josh/repos/mocha/test/integration/helpers.js
5:12 error "fs-extra" should be replaced with an alternative package. Read more here: https://github.com/es-tooling/module-replacements/blob/main/docs/modules/fs-extra.md depend/ban-dependencies
/Users/josh/repos/mocha/test/integration/options/watch.spec.js
3:12 error "fs-extra" should be replaced with an alternative package. Read more here: https://github.com/es-tooling/module-replacements/blob/main/docs/modules/fs-extra.md depend/ban-dependencies
Per https://github.com/es-tooling/module-replacements/blob/main/docs/modules/fs-extra.md:
fs-extra
adds some file system methods that previously were not included in the nativefs
module and addedPromise
support to thefs
methods.
Many of the fs-extra modules have since been implemented natively by Node.
Proposal: let's switch to native node:fs
/ node:fs/promises
.
Additional Info
Note that both are within test/integration
. This shouldn't impact users.
Metadata
Metadata
Assignees
Labels
area: repository toolingconcerning ease of contributionconcerning ease of contributionstatus: accepting prsMocha can use your help with this one!Mocha can use your help with this one!