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.

Sign up for GitHub

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

Jump to bottom

Bundle Pnpm with image #1755

Open
weyert opened this issue Aug 5, 2022 · 2 comments
Open

Bundle Pnpm with image #1755

weyert opened this issue Aug 5, 2022 · 2 comments

Comments

@weyert
Copy link

weyert commented Aug 5, 2022

Problem

Yarn gets prebundled with the Node docker image but another popular package manager PNPM is not

Solution

Ensure that PNPM gets bundled with the Node docker image

Alternatives to Consider

Drop the other third-party package manager Yarn
dunglas added a commit to dunglas/docker-node that referenced this issue Sep 7, 2022
@dunglas
feat: enable Corepack (pnpm support, unbundle yarn)
acfcf89 
This patch enables Corepack, allowing to use pnpm directly
and unbundling yarn from the default image.

Removing yarn also simplifies the image and the maintainance.

This change as been discussed in
nodejs#777.

Closes nodejs#777, nodejs#1645, nodejs#1755.
dunglas added a commit to dunglas/docker-node that referenced this issue Sep 7, 2022
@dunglas
feat: enable Corepack (pnpm support, unbundle yarn)
ea37959 
This patch enables Corepack, allowing to use pnpm directly
and unbundling yarn from the default image.

Removing yarn also simplifies the image and the maintenance.

This change has been discussed in
nodejs#777.

Closes nodejs#777, nodejs#1645, nodejs#1755.
@dunglas dunglas mentioned this issue Sep 7, 2022
Open
12 tasks
dunglas added a commit to dunglas/docker-node that referenced this issue Sep 7, 2022
@dunglas
feat: enable Corepack (pnpm support, unbundle yarn)
18da8a7 
This patch enables Corepack, allowing to use pnpm directly
and unbundling yarn from the default image.

Removing yarn also simplifies the image and the maintenance.

This change has been discussed in
nodejs#777.

Closes nodejs#777, nodejs#1645, nodejs#1755.
@gkatsanos

This comment was marked as spam.

Sign in to view
@alfaproject

This comment was marked as spam.

Sign in to view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@weyert @alfaproject @gkatsanos and others