Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a recursive option to mkdir #729

Merged
merged 19 commits into from
Nov 1, 2023
Merged

Add a recursive option to mkdir #729

merged 19 commits into from
Nov 1, 2023

Conversation

bcomnes
Copy link
Contributor

@bcomnes bcomnes commented Oct 26, 2023

This adds a recursive option to mkdir.

Closes #714

@bcomnes bcomnes self-assigned this Oct 26, 2023
@bcomnes bcomnes requested a review from jwerle October 26, 2023 22:50
api/fs/index.js Outdated Show resolved Hide resolved
@bcomnes bcomnes force-pushed the bret/recursive-mkdirp branch from 001c7e1 to c209d34 Compare October 27, 2023 23:19
src/core/core.hh Outdated Show resolved Hide resolved
src/core/fs.cc Outdated Show resolved Hide resolved
src/ipc/bridge.cc Outdated Show resolved Hide resolved
@bcomnes bcomnes force-pushed the bret/recursive-mkdirp branch 2 times, most recently from ca9f271 to eb868d5 Compare October 30, 2023 20:00
src/ipc/bridge.cc Outdated Show resolved Hide resolved
@bcomnes bcomnes force-pushed the bret/recursive-mkdirp branch from 2fa0214 to 155c91d Compare October 30, 2023 22:49
src/ipc/bridge.cc Outdated Show resolved Hide resolved
@bcomnes bcomnes force-pushed the bret/recursive-mkdirp branch from 48e06a3 to 4a87361 Compare October 31, 2023 21:41
@bcomnes bcomnes marked this pull request as ready for review October 31, 2023 21:44
@bcomnes bcomnes force-pushed the bret/recursive-mkdirp branch from 6d66880 to beb332a Compare October 31, 2023 21:52
src/core/fs.cc Outdated Show resolved Hide resolved
@bcomnes
Copy link
Contributor Author

bcomnes commented Oct 31, 2023

Ok, I think this should be good to go.

@bcomnes bcomnes requested review from jwerle and chicoxyzzy November 1, 2023 18:37
Copy link
Member

@chicoxyzzy chicoxyzzy left a comment

Choose a reason for hiding this comment

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

LGTM 👍
tests pass on my computer

@bcomnes
Copy link
Contributor Author

bcomnes commented Nov 1, 2023

Looking at CI failures a little closer.

@bcomnes
Copy link
Contributor Author

bcomnes commented Nov 1, 2023

The recursive tests are passing now. The fs.writeFile isn't working on master so I'll merge this now.

@bcomnes bcomnes merged commit a94db45 into master Nov 1, 2023
3 of 8 checks passed
@bcomnes bcomnes deleted the bret/recursive-mkdirp branch November 1, 2023 22:09
@jwerle jwerle added enhancement New feature or request javascript An issue, discussion, or task related to the runtime javascript api An issue, task, or discussion related to public runtime APIs labels Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api An issue, task, or discussion related to public runtime APIs enhancement New feature or request javascript An issue, discussion, or task related to the runtime javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement mkdir recursive
3 participants