Skip to content

pnpm-migration#711

Open
012xx wants to merge 4 commits intomainfrom
pnpm-migration
Open

pnpm-migration#711
012xx wants to merge 4 commits intomainfrom
pnpm-migration

Conversation

@012xx
Copy link
Copy Markdown
Contributor

@012xx 012xx commented Apr 15, 2026

Warning

Merge after #710.

I have replaced the parts using npm with pnpm.
The CI is failing, but since I am fixing it in 00d64ae, I believe it will pass if you rebase and rerun after #710 is merged.

@012xx 012xx self-assigned this Apr 15, 2026
@012xx 012xx marked this pull request as ready for review April 15, 2026 10:54
@012xx 012xx requested a review from takahiro-yamada April 15, 2026 10:55
Copy link
Copy Markdown

@takahiro-yamada takahiro-yamada left a comment

Choose a reason for hiding this comment

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

Can you replace pnpm install with pnpm install --frozen-lockfile in the other files too?

Comment thread e2e/manifests/manager/build-gatsby.sh Outdated
Comment thread config/samples/config.yaml Outdated
@012xx 012xx requested a review from takahiro-yamada April 15, 2026 22:26
Copy link
Copy Markdown

@takahiro-yamada takahiro-yamada left a comment

Choose a reason for hiding this comment

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

Check this please.
#711 (comment)

@012xx 012xx requested a review from takahiro-yamada April 20, 2026 04:51
Comment thread config/samples/config.yaml Outdated
Comment thread e2e/manifests/manager/build-gatsby.sh Outdated
@012xx 012xx force-pushed the pnpm-migration branch from 03c860a to d83f451 Compare May 1, 2026 06:33
@012xx 012xx marked this pull request as ready for review May 1, 2026 06:41
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.

3 participants