Skip to content

Conversation

@chenjiahan
Copy link
Member

@chenjiahan chenjiahan commented Feb 4, 2025

Summary

Fix corepack key id mismatch, see web-infra-dev/rslib#728, nodejs/corepack#612 and pnpm/pnpm#9029 for background.

This PR should fix all the failed CI, such as https://github.com/web-infra-dev/rspack/actions/runs/13118033695/job/36638142714?pr=9152

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Feb 4, 2025
@netlify
Copy link

netlify bot commented Feb 4, 2025

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit d773ab3
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67a2f6acdc34430008a8ae0e

@codspeed-hq
Copy link

codspeed-hq bot commented Feb 4, 2025

CodSpeed Performance Report

Merging #9153 will not alter performance

Comparing fix_corepack_ci_0204 (d773ab3) with main (b9e7842)

Summary

✅ 6 untouched benchmarks

@chenjiahan chenjiahan enabled auto-merge (squash) February 5, 2025 05:57
@chenjiahan chenjiahan merged commit e091d1c into main Feb 5, 2025
30 checks passed
@chenjiahan chenjiahan deleted the fix_corepack_ci_0204 branch February 5, 2025 05:58
ahabhgk pushed a commit that referenced this pull request Feb 6, 2025
* ci: fix corepack key id mismatch

* fix: pnpm

* fix: pnpm

* chore: echo

* fix: pin

* fix: try remove  --force

* Revert "fix: try remove  --force"

This reverts commit ac809f4.

* fix: try uninstall

* fix: ci

* fix: do not use corepack

* Revert "fix: do not use corepack"

This reverts commit e9024ac.

* fix: try another

* Revert "fix: try another"

This reverts commit 4f71800.

* fix: location

* fix: windows corepack not work

* ci: remove unused code

---------

Co-authored-by: jerrykingxyz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team The issue/pr is created by the member of Rspack.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants