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.

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

learn(fs): refresh #7230

Open
AugustinMauroy opened this issue Nov 15, 2024 · 1 comment
Open

learn(fs): refresh #7230

AugustinMauroy opened this issue Nov 15, 2024 · 1 comment
Labels
content Issues/pr concerning content help wanted learn Issues/pr concerning the learn section

Comments

@AugustinMauroy
Copy link
Member

IMO, the part about the FS is not quite right and there are missing parts.

  • The title of subsection "Manipulating Files" should be something like "working with file system" because the content is just not about file it's also about folder.
  • Adding a intorudction of fs/fs/promises + what mean file system ... like introduction of test runner
  • Include glob paterns
  • more examples with cjs/esm

Remaining question:

  • Should we put ESM example as first ?
  • Should we put fs/promises as first.
  • How to handle difference between fs and fs/promises
@AugustinMauroy AugustinMauroy added content Issues/pr concerning content learn Issues/pr concerning the learn section labels Nov 15, 2024
@AugustinMauroy
Copy link
Member Author

cc @nodejs/fs What is your opinion on that ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
content Issues/pr concerning content help wanted learn Issues/pr concerning the learn section
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ovflowd @AugustinMauroy