Skip to content

chore(devextreme): replace gulp js-bundles-prod with nx bundle executor#32806

Open
chaosmirage wants to merge 2 commits into26_1from
chore/migrate-devextreme-to-nx-part-6
Open

chore(devextreme): replace gulp js-bundles-prod with nx bundle executor#32806
chaosmirage wants to merge 2 commits into26_1from
chore/migrate-devextreme-to-nx-part-6

Conversation

@chaosmirage
Copy link
Contributor

No description provided.

@chaosmirage chaosmirage requested a review from a team March 5, 2026 15:17
@chaosmirage chaosmirage self-assigned this Mar 5, 2026
@chaosmirage chaosmirage marked this pull request as ready for review March 5, 2026 16:02
Copilot AI review requested due to automatic review settings March 5, 2026 16:02
Copy link
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

This PR migrates DevExtreme’s production JS bundle step away from the Gulp js-bundles-prod task to the devextreme-nx-infra-plugin:bundle Nx executor, aligning bundling with the monorepo’s Nx-based build pipeline.

Changes:

  • Adjusts the Nx bundle executor’s webpack config assembly (createWebpackConfig).
  • Reworks packages/devextreme Nx targets to build prod bundles via bundle:prod:build and apply license headers via a unified bundle:headers target (with a prod configuration).
  • Updates bundle:prod to run Nx commands instead of gulp js-bundles-prod.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
packages/nx-infra-plugin/src/executors/bundle/executor.ts Changes webpack optimization handling for bundles produced by the Nx executor.
packages/devextreme/project.json Replaces the Gulp prod bundling command with Nx bundle + license-header steps; refactors header target naming/configurations.

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.

2 participants