Skip to content

bug: enable Corepack before the i18n job resolves yarn - #558

Merged
lukepolo merged 1 commit into
mainfrom
bug/i18n-workflow-corepack
Aug 19, 2026
Merged

bug: enable Corepack before the i18n job resolves yarn#558
lukepolo merged 1 commit into
mainfrom
bug/i18n-workflow-corepack

Conversation

@lukepolo

Copy link
Copy Markdown
Contributor

No description provided.

The job has never gone green. `actions/setup-node`'s caching step shells
out to yarn to locate the cache folder, and the runner's global yarn is
1.22, which refuses to run at all against this repo's `packageManager:
yarn@4.11.0` — so the job died before installing anything.
Enable Corepack first. `corepack enable` writes its shims next to
whichever node is on PATH, and only the tool-cache node that setup-node
installs is writable without sudo, so setup-node has to run once before
Corepack and again after it for the cache. The bootstrap pass sets
package-manager-cache:false — from v5 setup-node caches automatically
whenever package.json has a `packageManager` field, so otherwise it
probes yarn too and fails in the same place.
Also bumps checkout/setup-node to v5, which run natively on Node 24 and
drop the Node 20 deprecation notice.
@lukepolo
lukepoloforce-pushed the bug/i18n-workflow-corepack branch from 008a4a5 to 26392deCompareAugust 19, 2026 01:17
@lukepolo
lukepolo merged commit 6afacd4 into mainAug 19, 2026
1 check passed
@lukepolo
lukepolo deleted the bug/i18n-workflow-corepack branch August 19, 2026 01:19
gabrielsdelima75 added a commit to gabrielsdelima75/web that referenced this pull request Aug 19, 2026
gabrielsdelima75 added a commit to gabrielsdelima75/web that referenced this pull request Aug 19, 2026
gabrielsdelima75 added a commit to gabrielsdelima75/web that referenced this pull request Aug 19, 2026
lukepolo pushed a commit that referenced this pull request Aug 19, 2026
Sign up for freeto 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

@lukepolo