Convert to npm#66
Conversation
socket fix --all --range-style preserve --minimum-release-age 7d --no-major-updates
|
CI status: Travis is red, failing at the install step (Bugbot is clean). Verified from the build log: Same root cause as EdgeApp/react-native-piratechain#21 — bumping |
|
Applied in 49aff36 — bumped |
j0ntz
left a comment
There was a problem hiding this comment.
Approving — Travis is green after the Node 18 bump. Flagging that this one-line CI change was made by me, so it's worth a quick re-review on your end.
CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
noneDescription
noneNote
Low Risk
Only package-manager and Travis config changes; no runtime or library code in the diff.
Overview
Migrates the repo from Yarn to npm for installs and CI, without changing application source in this diff.
.travis.ymlnow uses Node 18 only (drops 12), runsnpm cion install, andnpm run lint/npm run prepareinstead of the Yarn equivalents..npmrcis added withlegacy-peer-deps=trueandignore-scripts=true(replacing.yarnrc, which is removed)..gitignoreno longer ignorespackage-lock.json, so the npm lockfile can be tracked.Reviewed by Cursor Bugbot for commit 49aff36. Bugbot is set up for automated code reviews on this repo. Configure here.