Skip to content

Conversation

@aduh95
Copy link
Contributor

@aduh95 aduh95 commented Jan 22, 2024

This implements the flag discussed in #354 (comment). This of course makes sense only if we add support for arbitrary URLs, but that's for another PR.

@aduh95 aduh95 marked this pull request as ready for review February 21, 2024 04:29
@aduh95 aduh95 requested review from arcanis and merceyz February 21, 2024 05:07
@arcanis
Copy link
Contributor

arcanis commented Feb 21, 2024

Syntax-wise, what do you think of instead supporting corepack use yarn@npm:..., similar to how deps can be aliased?

mirqtio added a commit to mirqtio/anthrasite.io that referenced this pull request Oct 14, 2025
…n Node 20.16.0

The baseline allowlist workflow was failing with:
  Error: Cannot find matching keyid

Root cause: Node 20.16.0's bundled corepack has a signature verification bug
when running `corepack enable` that tries to auto-detect and install pnpm.

Fix: Add `corepack prepare pnpm@9 --activate` before pnpm install in both jobs
(gen-matrix and run-allowlisted) to explicitly install pnpm 9 without relying
on corepack's buggy auto-detection.

This is a known issue: nodejs/corepack#358

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
mirqtio added a commit to mirqtio/anthrasite.io that referenced this pull request Oct 14, 2025
…16.0 bug

The workflow was failing with corepack signature verification errors:
  Internal Error: Cannot find matching keyid

Root cause: Node 20.16.0's bundled corepack has a known bug with signature
verification that affects both `corepack enable` and `corepack prepare`.

Fix: Replace corepack usage with pnpm/action-setup@v4 action which installs
pnpm 9 directly, bypassing corepack entirely. This is the recommended
approach for CI environments.

Related: nodejs/corepack#358

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Sign up for free to 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.

3 participants