Skip to content

fix: bump unplugin to v2 to make plugin work under Node 26#10152

Merged
reidbarber merged 2 commits into
adobe:mainfrom
bhovhannes:main-hb-bump-unplugin
Jun 4, 2026
Merged

fix: bump unplugin to v2 to make plugin work under Node 26#10152
reidbarber merged 2 commits into
adobe:mainfrom
bhovhannes:main-hb-bump-unplugin

Conversation

@bhovhannes
Copy link
Copy Markdown
Contributor

Closes

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

unplugin@^1.4.0 uses rmdirSync method which is deprecated in Node 24 (emits a warning when called) and removed in Node 26.
That was fixed in unplugin@^2.

In this PR I bumped to unplugin^2.3.11. The latest version - unplugin@3.0.0, dropped CJS support, so migrating to it may mean more changes. In any case this PR does not hurt, migrating to unplugin@3.0.0 can be done separately.

📝 Test Instructions:

pipeline should pass

🧢 Your Project:

Adobe Workfront

@reidbarber reidbarber added this pull request to the merge queue Jun 4, 2026
Merged via the queue into adobe:main with commit bc141b2 Jun 4, 2026
30 of 31 checks passed
@bhovhannes bhovhannes deleted the main-hb-bump-unplugin branch June 4, 2026 17:57
@yihuiliao yihuiliao added the no testing Does not require manual testing during testing session label Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no testing Does not require manual testing during testing session

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants