Skip to content

add node:fs and node:fs/promises stub methods #3796

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

Merged
merged 1 commit into from
Jun 2, 2025
Merged

Conversation

anonrig
Copy link
Member

@anonrig anonrig commented Mar 25, 2025

Work in progress.

  • Add sync and callback APIs
  • Put it behind an experimental flag

@anonrig anonrig requested a review from jasnell March 25, 2025 16:59
@anonrig anonrig requested review from a team as code owners March 25, 2025 16:59
@anonrig anonrig requested a review from harrishancock March 25, 2025 16:59
@anonrig anonrig force-pushed the yagiz/add-fs-stub branch 2 times, most recently from cace91d to 719b4eb Compare March 26, 2025 20:25
@anonrig anonrig force-pushed the yagiz/add-fs-stub branch from 719b4eb to 199e223 Compare March 27, 2025 20:06
Copy link

github-actions bot commented Mar 27, 2025

The generated output of @cloudflare/workers-types matches the snapshot in types/generated-snapshot 🎉

@anonrig anonrig force-pushed the yagiz/add-fs-stub branch 3 times, most recently from 4f61aac to f0f809b Compare March 27, 2025 20:57
@anonrig anonrig requested a review from jasnell March 27, 2025 20:57
@anonrig anonrig force-pushed the yagiz/add-fs-stub branch 2 times, most recently from 183179e to 02dc23f Compare March 28, 2025 16:14
@anonrig anonrig force-pushed the yagiz/add-fs-stub branch from 02dc23f to 7e62dfd Compare April 17, 2025 13:26
jasnell
jasnell previously approved these changes Apr 17, 2025
@anonrig anonrig force-pushed the yagiz/add-fs-stub branch from 7e62dfd to c25e24a Compare April 21, 2025 16:33
@anonrig anonrig marked this pull request as draft April 25, 2025 23:57
@jasnell jasnell dismissed their stale review April 26, 2025 15:35

Will need more work

@jasnell

This comment was marked as outdated.

@anonrig

This comment was marked as outdated.

@jasnell
Copy link
Collaborator

jasnell commented May 14, 2025

PR updated with a range of APIs implemented. More to come.

@jasnell jasnell force-pushed the yagiz/add-fs-stub branch 8 times, most recently from 82c9853 to 68ab24b Compare May 20, 2025 20:13
Copy link
Contributor

@guybedford guybedford left a comment

Choose a reason for hiding this comment

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

Made an initial pass. Great start! The bigger comments about path encoding and errors can certainly be follow-ons, but I also think it's important to get these right early too.

@jasnell jasnell force-pushed the yagiz/add-fs-stub branch from 2ac6d2a to 64b7eab Compare June 2, 2025 17:59
@jasnell jasnell marked this pull request as ready for review June 2, 2025 18:03

This comment was marked as outdated.

@jasnell
Copy link
Collaborator

jasnell commented Jun 2, 2025

There is still a ton of work to do around this but I'd like to get this bit in then keep iterating in another PR.

@jasnell jasnell force-pushed the yagiz/add-fs-stub branch from 64b7eab to 7c13182 Compare June 2, 2025 18:12
@jasnell jasnell force-pushed the yagiz/add-fs-stub branch from 7c13182 to 9a95c42 Compare June 2, 2025 19:22
@jasnell jasnell merged commit 24d5823 into main Jun 2, 2025
18 checks passed
@jasnell jasnell deleted the yagiz/add-fs-stub branch June 2, 2025 20:00
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