fix(ci): fix pnpm installation #62
Merged
Conversation
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
The build script (tools/build.mjs) requires pnpm to be available when executing `pnpm install` and `pnpm bazel build` commands. Previously, pnpm was being set up after npm ci and before the build, which caused the build to fail because pnpm was not available in time. Changes: - Move "Setup pnpm" step to run immediately after "Setup Node JS" - Ensures pnpm is available before any build commands are executed
ricardochl
force-pushed
the
fix-pnpm-installation-ci
branch
from
c7f27af to
ff7dd56
Compare
Splaktar
approved these changes
Oct 27, 2025
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.
El script de build (tools/build.mjs) requiere que pnpm esté disponible
al ejecutar los comandos
pnpm installypnpm bazel build.Anteriormente, pnpm se configuraba después de npm ci y antes del build,
lo que causaba que el build fallara porque pnpm no estaba disponible a tiempo.
Cambios:
Esto corrige el fallo de build donde los comandos pnpm en la función
buildADEV() no podían ejecutarse.