Skip to content

fix(next-transpile-modules): use pnpm for pretest setup - #300

Merged
riceharvest merged 1 commit into
mainfrom
fix/next-transpile-pretest-pnpm-20260602-192417
Jun 2, 2026
Merged

fix(next-transpile-modules): use pnpm for pretest setup#300
riceharvest merged 1 commit into
mainfrom
fix/next-transpile-pretest-pnpm-20260602-192417

Conversation

@riceharvest

Copy link
Copy Markdown
Owner

Summary

  • Switches next-transpile-modules pretest setup from npm to pnpm so pnpm workspace config is not passed through npm.
  • Adds a regression test for the package-local pretest script.

Tests

  • corepack pnpm@9.6.0 --filter @opensourceframework/next-transpile-modules test
  • corepack pnpm@9.6.0 --filter @opensourceframework/next-transpile-modules run typings
  • corepack pnpm@9.6.0 exec eslint packages/next-transpile-modules/test/package-scripts.test.ts
  • corepack pnpm@9.6.0 exec prettier --check packages/next-transpile-modules/test/package-scripts.test.ts
  • git diff --check

@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@riceharvest
riceharvest merged commit c41e877 into main Jun 2, 2026
11 checks passed
@riceharvest
riceharvest deleted the fix/next-transpile-pretest-pnpm-20260602-192417 branch June 2, 2026 19:34
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