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(package manager): introduce Corepack #674

Merged
merged 10 commits into from
Feb 26, 2023

Conversation

sounisi5011
Copy link
Owner

@sounisi5011 sounisi5011 commented Feb 26, 2023

From now on, we will use Corepack to manage pnpm versions.
see https://nodejs.org/api/corepack.html
see https://github.com/nodejs/corepack

Corepack automatically prepares the pnpm.
Thus, all we have to do is execute the `corepack enable` command.
Surprisingly, Corepack seems to work with all versions of Node.js on GitHub Actions.
Perhaps there is no need to run the `corepack enable` command.
…le` command

The following error occurred on each job:

    pnpm: command not found

It seems that the `corepack enable` command must be executed.

This reverts commit 8cb0f48.
…ore understandable

The step name "Setup Corepack" indicates that it is setting up Corepack.
However, this step name does not clearly indicate that pnpm will be available automatically.
Contributors who are not familiar with Node.js will not be able to grasp what is possible by enabling Corepack.
So we fix the step names and clarify what we are enabling Corepack for.
@codeclimate
Copy link

codeclimate bot commented Feb 26, 2023

Code Climate has analyzed commit f1cdfd5 and detected 0 issues on this pull request.

View more on Code Climate.

@sounisi5011 sounisi5011 marked this pull request as ready for review February 26, 2023 17:15
@sounisi5011 sounisi5011 merged commit 2a5ca2d into main Feb 26, 2023
@sounisi5011 sounisi5011 deleted the package-manager/introduce-corepack branch February 26, 2023 17:17
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.

1 participant