Skip to content

ci: migrate from yarn to pnpm#163

Open
n33pm wants to merge 1 commit intokyeotic:mainfrom
n33pm:switch-to-pnpm
Open

ci: migrate from yarn to pnpm#163
n33pm wants to merge 1 commit intokyeotic:mainfrom
n33pm:switch-to-pnpm

Conversation

@n33pm
Copy link
Copy Markdown
Contributor

@n33pm n33pm commented Mar 10, 2026

Summary

  • Replace yarn install with pnpm install in CI workflows (test.yaml and size.yml)
  • Add pnpm/action-setup@v4 step to both workflows to install pnpm v10
  • Enable pnpm cache via actions/setup-node cache setting
  • Remove package-lock.json (no longer needed with pnpm)

@n33pm n33pm requested a review from kyeotic as a code owner March 10, 2026 08:19
@n33pm n33pm force-pushed the switch-to-pnpm branch 4 times, most recently from f020ee0 to e69730d Compare March 10, 2026 08:28
@kyeotic
Copy link
Copy Markdown
Owner

kyeotic commented Mar 10, 2026

I just dropped yarn for default npm in #162. pnpm was running into issues with jest, I didn't want to deal with it

@n33pm n33pm force-pushed the switch-to-pnpm branch 5 times, most recently from d763a1a to 3a24b5d Compare March 13, 2026 16:08
fix: add @types/testing-library__jest-dom as direct devDependency for pnpm strict isolation
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.

2 participants