Skip to content

chore: setup unified lockfile with PNPM workspaces#8438

Draft
pearigee wants to merge 16 commits into
mainfrom
pearigee-unified-lockfile
Draft

chore: setup unified lockfile with PNPM workspaces#8438
pearigee wants to merge 16 commits into
mainfrom
pearigee-unified-lockfile

Conversation

@pearigee
Copy link
Copy Markdown
Contributor

@pearigee pearigee commented Jun 3, 2026

A prototype experimenting with a single lockfile for the whole respository using PNPM workspaces.

@pearigee pearigee requested review from a team as code owners June 3, 2026 23:01
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request configures a pnpm workspace by adding a pnpm-workspace.yaml file, updating .gitignore files across the repository to ignore lockfiles globally, and modifying the CI script ci/run_single_test.sh to run compilation instead of installing dependencies. Feedback on the CI script suggests using the --filter ...{.} option with pnpm run compile to ensure that local workspace dependencies are compiled in the correct topological order.

Comment thread ci/run_single_test.sh Outdated
@pearigee pearigee force-pushed the pearigee-unified-lockfile branch from 12993e3 to 4d6ede9 Compare June 3, 2026 23:03
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@pearigee pearigee changed the title Setup unified lockfile with pnpm workspaces chore: setup unified lockfile with PNPM workspaces Jun 3, 2026
@pearigee pearigee marked this pull request as draft June 3, 2026 23:05
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