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

Replace yarn with pnpm #4828

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Replace yarn with pnpm #4828

wants to merge 1 commit into from

Conversation

MaxLeiter
Copy link
Member

@MaxLeiter MaxLeiter commented Jan 27, 2024

It's time to move on from yarn 1.

Not that I contribute enough to care much, but it was frustrating to use yarn while setting my local repo up again. Happy to go back to npm / upgrade to a newer version of yarn too.

It's time to move on from yarn 1, imo.
Not that I contribute enough to care, but it was frustrating to use yarn
while setting my local repo up again.
@brunnre8
Copy link
Member

Hrm...
This also needs changes for the package logic within TL where yarn is used as well.

This also needs corresponding PRs to the deb package generation as well.

As is this is incomplete and not mergable I think

@brunnre8
Copy link
Member

brunnre8 commented Jan 28, 2024

Fwiw I have 0 reasons not to change from yarn.

Don't think npm is much better though, both are shitty (or at least were historically).

What made you go for pnpm?

Copy link
Member

@brunnre8 brunnre8 left a comment

Choose a reason for hiding this comment

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

As already mentioned, this needs some additional work.
Marking as such so that the UI is easier to filter

@brunnre8 brunnre8 added the Status: changes-requested Review happened but some changes need to be made label Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: changes-requested Review happened but some changes need to be made
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants