Skip to content

chore(#787): migrate from yarn to npm#796

Merged
garethbowen merged 29 commits intomainfrom
move-to-npm
Apr 30, 2026
Merged

chore(#787): migrate from yarn to npm#796
garethbowen merged 29 commits intomainfrom
move-to-npm

Conversation

@garethbowen
Copy link
Copy Markdown
Collaborator

Closes #787

I have verified this PR works in these browsers (latest versions):

  • Chrome
  • Firefox
  • Safari (macOS)
  • Safari (iOS)
  • Chrome for Android
  • Not applicable

This is quite risky because unfortunately there was no way to convert back to npm while preserving the lock file so it required a clean npm install. I manually reverted some known libraries that a pegged to a version but I may have missed some. So we should regression test this fully before release, BUT moving WF to CF will also be risky and require regression testing so I suggest we defer it until the next minor Central release.

@garethbowen garethbowen requested a review from latin-panda April 29, 2026 23:36
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 29, 2026

⚠️ No Changeset found

Latest commit: 3d0a3fe

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@garethbowen garethbowen changed the title Move to npm chore(#787): migrate from yarn to npm Apr 29, 2026
Copy link
Copy Markdown
Collaborator

@latin-panda latin-panda left a comment

Choose a reason for hiding this comment

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

It looks good! I haven't done a deep test yet, but I did leave a couple of minor fixes below.

Comment thread README.md Outdated
Comment thread scripts/feature-matrix/render.js Outdated
@garethbowen
Copy link
Copy Markdown
Collaborator Author

@latin-panda I'd like to get this merged soon because if anything changes in the yarn.lock it's hard to port it to the package-lock.json. However it's fairly risky because of unknown changes in dependencies. However because we've just released now is the perfect time to merge and find any issues. So I'm going to go for it. Sorry in advance if I've missed anything!

@garethbowen garethbowen merged commit 5f7cd43 into main Apr 30, 2026
50 checks passed
@garethbowen garethbowen deleted the move-to-npm branch April 30, 2026 23:20
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.

Convert to using npm instead of yarn

2 participants