Skip to content

This PR updates the documentation for fs.statfs to clarify that bsize is in bytes. Fixes #50749#59407

Closed
AtCloudDiwash wants to merge 1 commit into
nodejs:mainfrom
AtCloudDiwash:fix-statfs-documentation
Closed

This PR updates the documentation for fs.statfs to clarify that bsize is in bytes. Fixes #50749#59407
AtCloudDiwash wants to merge 1 commit into
nodejs:mainfrom
AtCloudDiwash:fix-statfs-documentation

Conversation

@AtCloudDiwash

@AtCloudDiwash AtCloudDiwash commented Aug 8, 2025

Copy link
Copy Markdown

Summary

This PR updates the documentation for fs.statfs to clarify that bsize is measured in bytes.

Description

The current documentation for fs.statfs does not explicitly state the unit for bsize. This change clarifies that the value returned is in bytes to improve developer clarity and prevent implementation errors.

Fixes: #50749
Fixes: #59407

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system. labels Aug 8, 2025

@Hardanish-Singh Hardanish-Singh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@AtCloudDiwash

Copy link
Copy Markdown
Author

Hi @nodejs/fs, I'm checking back on this PR as it's been open for a few months. It's a small documentation fix to clarify the bsize unit in fs.statfs.

Could a collaborator please review if this is correct and qualified to be merged? I’m happy to make any necessary updates if needed. Thank you!

@aduh95

aduh95 commented May 27, 2026

Copy link
Copy Markdown
Contributor

Superseded by #62578

@aduh95 aduh95 closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Issues and PRs related to the documentations. fs Issues and PRs related to the fs subsystem / file system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Missing documentation in fs.StatFs

4 participants