Skip to content

Replace (p)npx with pnpm in publishing and workflow scripts#33262

Merged
alexslavr merged 1 commit into25_1from
lavrov/ci-no-npx-25_1
Apr 14, 2026
Merged

Replace (p)npx with pnpm in publishing and workflow scripts#33262
alexslavr merged 1 commit into25_1from
lavrov/ci-no-npx-25_1

Conversation

@alexslavr
Copy link
Copy Markdown
Contributor

Cherry-pick of #33212

@alexslavr alexslavr self-assigned this Apr 14, 2026
@alexslavr alexslavr requested a review from a team as a code owner April 14, 2026 13:23
@alexslavr alexslavr requested a review from Copilot April 14, 2026 13:23
@alexslavr alexslavr changed the title Replace (p)npx with pnpm in publishing and workflow scripts (#33212) Replace (p)npx with pnpm in publishing and workflow scripts Apr 14, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

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.json scripts for change-package-scope and make-artifacts-package (both ts-node entrypoints).
  • Replaced pnpx usage in multiple GitHub Actions workflows with pnpm exec (and pnpm --filter ... exec where needed).
  • Updated package publishing workflow to use pnpm run for 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).

Comment on lines +84 to +85
npx --no-install devextreme-bundler-init custom-bundle
npx --no-install devextreme-bundler custom-bundle No newline at end of file
Copy link

Copilot AI Apr 14, 2026

Choose a reason for hiding this comment

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

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.

Copilot uses AI. Check for mistakes.
@alexslavr alexslavr merged commit 0281317 into 25_1 Apr 14, 2026
103 of 104 checks passed
@alexslavr alexslavr deleted the lavrov/ci-no-npx-25_1 branch April 14, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants