Skip to content

Add promise support for missing fs methods #1045

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 2 commits into from
Jan 14, 2025
Merged

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Jan 10, 2025

Fixes #1044

Somehow, I added a comment about fs.cp but never added it to the list πŸ€·β€β™‚οΈ

@RyanZim
Copy link
Collaborator Author

RyanZim commented Jan 10, 2025

CI is failing because macos-latest now runs on arm64, but old Node versions only support x64 arch

@RyanZim RyanZim force-pushed the ryan/missing-fs-methods branch from ed11fe2 to 58c8d91 Compare January 10, 2025 20:30
@RyanZim
Copy link
Collaborator Author

RyanZim commented Jan 10, 2025

Temporarily fixed the macOS version to work around this

@JPeer264
Copy link
Collaborator

CI is failing because macos-latest now runs on arm64, but old Node versions only support x64 arch

For that it would make sense to drop the support for 14.x and 16.x, and start supporting newer versions, at least in GitHub Actions. WDYT?

@RyanZim
Copy link
Collaborator Author

RyanZim commented Jan 14, 2025

Yeah, we should drop old Node versions, but that'd be a separate breaking change, and I'd like to do some thinking about other breaking changes we should include in the major release.

@RyanZim RyanZim merged commit e75b9bf into master Jan 14, 2025
12 checks passed
@RyanZim RyanZim deleted the ryan/missing-fs-methods branch January 14, 2025 16:52
jelmore1674 added a commit to jelmore1674/build-changelog that referenced this pull request Mar 19, 2025
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [fs-extra](https://github.com/jprichardson/node-fs-extra) | devDependencies | minor | [`11.2.0` -> `11.3.0`](https://renovatebot.com/diffs/npm/fs-extra/11.2.0/11.3.0) |

---

### Release Notes

<details>
<summary>jprichardson/node-fs-extra (fs-extra)</summary>

### [`v11.3.0`](https://github.com/jprichardson/node-fs-extra/blob/HEAD/CHANGELOG.md#1130--2025-01-15)

[Compare Source](jprichardson/node-fs-extra@11.2.0...11.3.0)

-   Add promise support for newer `fs` methods ([#&#8203;1044](jprichardson/node-fs-extra#1044), [#&#8203;1045](jprichardson/node-fs-extra#1045))
-   Use `fs.opendir` in `copy()`/`copySync()` for better perf/scalability ([#&#8203;972](jprichardson/node-fs-extra#972), [#&#8203;1028](jprichardson/node-fs-extra#1028))

</details>

---

### Configuration

πŸ“… **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

β™» **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

πŸ”• **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOS4xMDcuMiIsInVwZGF0ZWRJblZlciI6IjM5LjEwNy4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Reviewed-on: https://git.justinelmore.dev/jelmore1674/build-changelog/pulls/78
Co-authored-by: Renovate Bot <[email protected]>
Co-committed-by: Renovate Bot <[email protected]>

chore: update changelog
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.

Support for statfs() and other newer fs functions in more recent versions of node.js
2 participants