Replace (p)npx with pnpm in publishing and workflow scripts#33262
Replace (p)npx with pnpm in publishing and workflow scripts#33262
Conversation
There was a problem hiding this comment.
Pull request overview
Updates CI/publishing workflows to prefer pnpm execution patterns over (p)npx, aligning workflow command execution with the repo’s pnpm-managed monorepo setup (cherry-pick of #33212).
Changes:
- Added root
package.jsonscripts forchange-package-scopeandmake-artifacts-package(bothts-nodeentrypoints). - Replaced
pnpxusage in multiple GitHub Actions workflows withpnpm exec(andpnpm --filter ... execwhere needed). - Updated package publishing workflow to use
pnpm runfor the artifact packaging/scope-change steps.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Adds root scripts to run TypeScript publishing helpers via pnpm run. |
| .github/workflows/wrapper_tests.yml | Switches Puppeteer browser install from pnpx to pnpm --filter ... exec. |
| .github/workflows/themebuilder_tests.yml | Replaces pnpx calls for gulp/stylelint with pnpm exec. |
| .github/workflows/renovation.yml | Replaces pnpx gulp ... with pnpm exec gulp .... |
| .github/workflows/packages_publishing.yml | Uses pnpm run scripts instead of npx ts-node ... for publishing helpers. |
| .github/workflows/demos_visual_tests.yml | Replaces pnpx gulp ... with pnpm exec gulp ... for demos bundling config. |
| .github/workflows/build_all.yml | Adjusts bundler commands to npx --no-install (still uses npx). |
| npx --no-install devextreme-bundler-init custom-bundle | ||
| npx --no-install devextreme-bundler custom-bundle No newline at end of file |
There was a problem hiding this comment.
This workflow still uses npx (now with --no-install) even though the PR title/description says it replaces (p)npx with pnpm. To avoid the mismatch (and to make the step fully deterministic), either switch these invocations to a pnpm-based execution (pnpm exec ... after setting up pnpm in this job) or call the locally-installed binary directly (e.g., from node_modules/.bin) and update the PR title/description accordingly.
Cherry-pick of #33212