fs: fix enumerability of `Date` properties on `Stats` objects by LiviaMedeiros · Pull Request #63328 · nodejs/node · GitHub
Skip to content

fs: fix enumerability of Date properties on Stats objects#63328

Merged
LiviaMedeiros merged 1 commit into
nodejs:mainfrom
LiviaMedeiros:stats-date-enumerable
May 17, 2026
Merged

fs: fix enumerability of Date properties on Stats objects#63328
LiviaMedeiros merged 1 commit into
nodejs:mainfrom
LiviaMedeiros:stats-date-enumerable

Conversation

@LiviaMedeiros

Copy link
Copy Markdown
Member

This basically implements suggestions for Temporal version of these properties: #60789 (comment) (prefer setOwnProperty() in setter which also makes accessed properties enumerable), #60789 (comment) (prefer setOwnProperty() in getter to avoid triggering setter).

/cc @aduh95

@nodejs-github-bot nodejs-github-bot added fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module. labels May 15, 2026
@codecov

codecov Bot commented May 15, 2026

Copy link
Copy Markdown

@addaleax addaleax added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. request-ci Add this label to start a Jenkins CI on a PR. labels May 15, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 15, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: nodejs#63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@LiviaMedeiros LiviaMedeiros force-pushed the stats-date-enumerable branch from ecdc1fb to a9b0a83 Compare May 17, 2026 12:45
@LiviaMedeiros LiviaMedeiros merged commit a9b0a83 into nodejs:main May 17, 2026
20 of 21 checks passed
@LiviaMedeiros

Copy link
Copy Markdown
Member Author

aduh95 pushed a commit that referenced this pull request May 19, 2026
Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: #63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request May 19, 2026
Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: #63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request May 23, 2026
Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: #63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
araujogui pushed a commit to araujogui/node that referenced this pull request May 26, 2026
Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: nodejs#63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
aduh95 pushed a commit that referenced this pull request Jun 18, 2026
Signed-off-by: LiviaMedeiros <livia@cirno.name>
PR-URL: #63328
Reviewed-By: René <contact.9a5d6388@renegade334.me.uk>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fs Issues and PRs related to the fs subsystem / file system. needs-ci PRs that need a full CI run. util Issues and PRs related to the built-in util module.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants