meilisearch / meilisearch-js

JavaScript client for the Meilisearch API

Home Page:https://www.meilisearch.com

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update `yarn`

flevi29 opened this issue · comments

Currently this repository is using yarn classic.

One day, perhaps not too long from now, it will be deprecated for good, it says on the install page.

image

There are benefits to upgrading.

However the new version is to be managed through corepack.
In my opinion the best way forward is to change actions/setup-node@v4 so that it no longer installs yarn globally, but rather enables corepack. #1664 (comment)

Important

This will (theoretically, but I am 99% sure) not impact the rest of the JS repositories, as enabling corepack means that as soon as yarn or pnpm is entered into the command line, corepack installs it (if it isn't already installed), and then executes it. (I'm not sure why, but by default it installs Yarn classic, so it's fine)

Read more about upgrading to the latest Yarn:
https://yarnpkg.com/getting-started/install

Migration should be very simple:
https://yarnpkg.com/migration/guide#migration-steps

Okay, I had some misconceptions. I thought actions/setup-node was a Meilisearch custom script, but it's entirely out of your hands. The approach will be a little different it seems.

actions/setup-node#480
actions/setup-node#482
actions/setup-node#651

It seems like they're not entirely into Corepack, because it's experimental, meanwhile Yarn wants you to use Corepack, so I'm not sure what's going on here. Seems like no one can agree on anything, classic JavaScript community.