Skip to content

chore: restore packageManager field for corepack - #1429

Closed
jycouet wants to merge 1 commit into
sveltejs:mainfrom
jycouet:eco-sys
Closed

jycouet wants to merge 1 commit into
sveltejs:mainfrom
jycouet:eco-sys

Conversation

@jycouet

@jycouet jycouet commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

Description

Corepack only reads packageManager, not devEngines. Since #1375 removed it, svelte-ecosystem-ci (which runs pnpm through corepack) falls back to its own pnpm 11.22.0 and fails the devEngines check (run).

Restores packageManager and pins devEngines to the same exact version, which avoids the pnpm warning that #1375 was fixing.

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of sv from c12f998:

pnx https://pkg.svelte.dev/sv/c/c12f99800c01a44b1df1d0907b3fe28199726631 create

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1429

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Oct 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: c12f998

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

1 participant