Skip to content

chore: drop mkdirp dependency#1350

Merged
UlisesGascon merged 1 commit intoexpressjs:mainfrom
wojtekmaj:fs.mkdir
Aug 8, 2025
Merged

chore: drop mkdirp dependency#1350
UlisesGascon merged 1 commit intoexpressjs:mainfrom
wojtekmaj:fs.mkdir

Conversation

@wojtekmaj
Copy link
Copy Markdown
Contributor

fs.mkdir gained recursive option in Node.js v10.12.0 (source: https://nodejs.org/api/fs.html#fsmkdirpath-options-callback). Minimum version of Node.js supported by multer at the moment is v10.16.0, so we should be good to go.

fs.mkdir gained `recursive` option in Node.js v10.12.0 (source: https://nodejs.org/api/fs.html#fsmkdirpath-options-callback). Minimum version of Node.js supported by multer at the moment is v10.16.0, so we should be good to go.
Copy link
Copy Markdown
Member

@LinusU LinusU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Neat 👍

Copy link
Copy Markdown
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great one! Thanks @wojtekmaj 👏

@UlisesGascon UlisesGascon self-assigned this Aug 8, 2025
@UlisesGascon UlisesGascon merged commit 6bb3512 into expressjs:main Aug 8, 2025
18 checks passed
@wojtekmaj wojtekmaj deleted the fs.mkdir branch August 8, 2025 11:41
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 98.496% (-0.006%) from 98.502%
when pulling ed29390 on wojtekmaj:fs.mkdir
into 2e07e77 on expressjs:main.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants