Skip to content

Document the node:fs impl #24082

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Oct 16, 2025
Merged

Document the node:fs impl #24082

merged 2 commits into from
Oct 16, 2025

Conversation

@jasnell
Copy link
Contributor

@jasnell jasnell commented Jul 31, 2025 •
edited
Loading

Summary

Not quite ready to land as it's still experimental in workerd, but getting ready. Initial docs for node:fs

Screenshots (optional)

Documentation checklist

  • The documentation style guide has been adhered to.
  • If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
  • Files which have changed name or location have been allocated redirects.

@github-actions github-actions bot added size/s product:workers Related to Workers product labels Jul 31, 2025
mikenomitch
mikenomitch reviewed Aug 1, 2025
View reviewed changes
jasnell
jasnell commented Aug 1, 2025
View reviewed changes
jasnell
jasnell commented Aug 1, 2025
View reviewed changes
@vicb vicb mentioned this pull request Aug 5, 2025
Merged
6 tasks
jasnell
jasnell commented Aug 5, 2025
View reviewed changes
@vicb
Copy link
Contributor

vicb commented Aug 28, 2025

Not quite ready to land as it's still experimental in workerd, but getting ready. Initial docs for node:fs

@jasnell can we move this forward as the flag is no more experimental after workerd 20250827 ?

@jasnell jasnell marked this pull request as ready for review September 10, 2025 18:56
@jasnell jasnell requested review from mikenomitch and vicb September 10, 2025 18:56
@jasnell jasnell force-pushed the jasnell/nodefs branch 2 times, most recently from 5072dbb to 431a2ad Compare September 15, 2025 13:59
@jasnell jasnell mentioned this pull request Sep 15, 2025
Merged
jasnell
jasnell commented Sep 24, 2025
View reviewed changes
jasnell
jasnell commented Sep 24, 2025
View reviewed changes
@vicb vicb merged commit f8a23eb into cloudflare:production Oct 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@irvinebroque irvinebroque irvinebroque approved these changes

@GregBrimble GregBrimble Awaiting requested review from GregBrimble GregBrimble is a code owner

@WalshyDev WalshyDev Awaiting requested review from WalshyDev WalshyDev is a code owner

@aninibread aninibread Awaiting requested review from aninibread aninibread is a code owner

@korinne korinne Awaiting requested review from korinne korinne is a code owner

@kodster28 kodster28 Awaiting requested review from kodster28 kodster28 is a code owner

@mikenomitch mikenomitch Awaiting requested review from mikenomitch mikenomitch is a code owner

@vicb vicb Awaiting requested review from vicb

Assignees

@irvinebroque irvinebroque

@mikenomitch mikenomitch

@GregBrimble GregBrimble

@WalshyDev WalshyDev

@kodster28 kodster28

@korinne korinne

Labels

product:workers Related to Workers product size/s

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@jasnell @vicb @irvinebroque @mikenomitch @GregBrimble @WalshyDev @kodster28 @korinne