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.

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

Bump fs-extra to 9.x #441

Merged
merged 1 commit into from
Dec 29, 2022
Merged

Bump fs-extra to 9.x #441

merged 1 commit into from
Dec 29, 2022

Conversation

stianjensen
Copy link
Contributor

@stianjensen stianjensen commented Dec 19, 2022

This gets in some performance improvements, as well as upgrades underlying dependencies to newer versions.

Bumping to 9.x will require nodejs 10.x as a minimum version, which is
already the minimum version declared in package.json.

https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md

@orta
Copy link
Collaborator

orta commented Dec 28, 2022

Yeah, doing a lower version is better IMO - this isn't a good enough reason to force a semver major on everyone using patch-package

This gets in some performance improvements, as well as upgrades
underlying dependencies to newer versions.

Bumping to 9.x will require nodejs 10.x as a minimum version, which is
already the minimum version declared in package.json.

https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md
@stianjensen stianjensen changed the title Bump fs-extra to 11.x Bump fs-extra to 9.x Dec 28, 2022
@stianjensen
Copy link
Contributor Author

Changed to update to 9.x

@orta
Copy link
Collaborator

orta commented Dec 29, 2022

Looks good to me, thanks!

@orta orta merged commit 85f6eab into ds300:master Dec 29, 2022
@ds300
Copy link
Owner

ds300 commented Jan 3, 2023

Released in v6.5.1, thanks for the contribution 🎉 ❤️

@stianjensen stianjensen deleted the fs-extra-bump branch January 3, 2023 18:13
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.

3 participants