ci: use repo pnpm 10 (fixes lockfile-ignore + postman-code-generators…#1129
Merged
MeenakshiArunsankar merged 1 commit intoJun 29, 2026
Merged
Conversation
… postinstall crash)
824ff98
into
vicharanashala:main
1 of 2 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ci: align workflows to repo pnpm 10 (fix e2e install failure)
What
Switch the
nightly-staging-e2e,linter, anddeploy-frontendworkflows from a hard-pinned pnpm 8 to the repo's declared pnpm 10 (pnpm/action-setup@v4with no version pin, so it reads the rootpackageManager). This mirrors the already-passingjest-test.yml.Why
The e2e pipeline was failing at the Install Dependencies step. Running CI on pnpm 8 while the repo is on pnpm 10 (
packageManager: pnpm@10.12.1) caused two problems:Lockfile ignored —
WARN Ignoring not compatible lockfile … pnpm-lock.yaml→ non-deterministic, full re-resolve.Fatal postinstall crash — pnpm 8 runs every dependency's lifecycle script, including the broken
postman-code-generatorsone:pnpm 10 gates dependency build/postinstall scripts by default, so that script never runs — which is exactly why local installs (pnpm 10) already succeed. Aligning CI to pnpm 10 fixes both the lockfile warning and the crash.
Scope / safety
jest-test.yml,docs-check.yml,deploy-docs.ymlalready used pnpm 10 (unchanged).Testing
pnpm installon pnpm 10 completes successfully (the broken script is gated).