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

chore: migrate to npm #1165

Closed
wants to merge 4 commits into from
Closed

chore: migrate to npm #1165

wants to merge 4 commits into from

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Mar 1, 2022

Description

Migrate the repo to npm 8.

The main motivation for this move is to get better cacheability (more stable) than Yarn Classic, which randomly gets corrupted and have to be force invalidated. There is also a desire to move back to npm at some point when isolated mode gets implemented.

The alternative would be to upgrade to Yarn 3, which is less intrusive, but comes with its own set of issues.

Test plan

CI should pass.

@github-actions github-actions bot added the chore Improvements that don't directly affect features label Mar 1, 2022
@tido64 tido64 force-pushed the tido/npm branch 11 times, most recently from 489b930 to d9724d5 Compare March 3, 2022 10:05
@github-actions github-actions bot added the feature: align-deps This is related to align-deps label Mar 3, 2022
@tido64 tido64 force-pushed the tido/npm branch 3 times, most recently from e94d6e0 to 1e1a46a Compare March 3, 2022 10:45
packages/draft-patch-rnmacos/package.json Outdated Show resolved Hide resolved
packages/dep-check/package.json Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
@tido64
Copy link
Member Author

tido64 commented Apr 7, 2022

Moved the branch to my own fork. Will open a new PR if necessary.

@tido64 tido64 closed this Apr 7, 2022
@tido64 tido64 deleted the tido/npm branch April 7, 2022 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Improvements that don't directly affect features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants