Skip to content

Use pnpm; install after updates to upgrade the lockfile at the workspace#21158

Merged
kategengler merged 2 commits intomainfrom
kg-fix-template-gen
Mar 4, 2026
Merged

Use pnpm; install after updates to upgrade the lockfile at the workspace#21158
kategengler merged 2 commits intomainfrom
kg-fix-template-gen

Conversation

@kategengler
Copy link
Copy Markdown
Member

level

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 4, 2026

📊 Package size report   -0%↓

File Before (Size / Brotli) After (Size / Brotli)
Total (Includes all files) 17.3 MB / 3.2 MB -0%↓17.3 MB / -0%↓3.2 MB
Tarball size 3.9 MB -0%↓3.9 MB

🤖 This report was automatically generated by pkg-size-action

node-version: 20
- name: Setup pnpm
uses: pnpm/action-setup@v4
with:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this should not be needed as this action should pick up on the packageManager field from the root package.json, iirc

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

We have it elsewhere in the actions that do it

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

well I guess they're all wrong 🙈 cause locally we use pnpm 10.5 haha

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

ugh

@kategengler kategengler force-pushed the kg-fix-template-gen branch from b1201b6 to 3a08a23 Compare March 4, 2026 01:10
@kategengler kategengler merged commit 8958d5a into main Mar 4, 2026
31 checks passed
@kategengler kategengler deleted the kg-fix-template-gen branch March 4, 2026 01:40
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