- Issue created by @alexpott
- π¬π§United Kingdom alexpott πͺπΊπ
This is even more important when corepack becomes manadatory for developing on multiple core branches following π Update from Yarn 1 to Yarn 4 RTBC
- Status changed to Needs review
8 months ago 12:05pm 15 March 2024 - Status changed to RTBC
8 months ago 1:10pm 15 March 2024 - π¬π§United Kingdom longwave UK
While I couldn't reproduce this locally (I am on an older NodeJS/corepack setup than Alex) I think this is a good idea:
- Yarn will apparently warn if
packageManager
is set but corepack is not enabled, which will help users get it installed locally. - Once corepack is enabled it means everyone will use the specified version of yarn locally.
- Yarn will apparently warn if
- Status changed to Fixed
8 months ago 2:40pm 15 March 2024 - π¬π§United Kingdom alexpott πͺπΊπ
Committing because I cannot commit to Drupal until this is merged... lol
Committed and pushed ef8b6e30b8 to 11.x and 2c9040ee8f to 10.3.x and c13f63ce92 to 10.2.x. Thanks!
-
alexpott β
committed c13f63ce on 10.2.x
Issue #3428122 by alexpott, longwave: Add packageManager key to core/...
-
alexpott β
committed c13f63ce on 10.2.x
-
alexpott β
committed 2c9040ee on 10.3.x
Issue #3428122 by alexpott, longwave: Add packageManager key to core/...
-
alexpott β
committed 2c9040ee on 10.3.x
-
alexpott β
committed ef8b6e30 on 11.x
Issue #3428122 by alexpott, longwave: Add packageManager key to core/...
-
alexpott β
committed ef8b6e30 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Needs work
7 months ago 11:25pm 17 April 2024 - πΊπΈUnited States xjm
This broke my commit workflow and unfortunately took a bit of head-desking to figure out what was going on; I spent quite a bit of time tearing apart and rebuilding my toolchain before I understood the problem. From the yarn page about corepack:
DANGER
Some third-party distributors may not include Corepack by default, in particular if you install Node.js from your system package manager. If that happens, running npm install -g corepack before corepack enable should do the trick.I think this needs a change record and probably a handbook page update somewhere. I already updated https://github.com/alexpott/d8githooks/blob/main/README.md but that's not what 99.99% of people using core's build tooling locally will reference. :)
- πΊπΈUnited States xjm
In terms of change records we can probably just add to the main Yarn 4 one at https://www.drupal.org/node/3428571; β doing that now.
- πΊπΈUnited States xjm
Updated the CR; still needs to go in the handbook setup instructions somewhere though.
- Status changed to Fixed
4 months ago 2:28am 8 July 2024 - π³πΏNew Zealand quietone
This was include in Frontend Developer tools for Drupal core β in Sep 2022. I searched the Drupal wiki for other places this information needed to be included and could not find any other suitable page.
Therefor I am going to restore the fixed status. If someone has a better idea, re-open and add a comment.
Automatically closed - issue fixed for 2 weeks with no activity.