Skip to content

CI: Apply setName transformation to devextreme-dist-internal package.json#33780

Merged
alexslavr merged 1 commit into
26_1from
lavrov/fix-devextreme-dist-build-26_1
Jun 1, 2026
Merged

CI: Apply setName transformation to devextreme-dist-internal package.json#33780
alexslavr merged 1 commit into
26_1from
lavrov/fix-devextreme-dist-build-26_1

Conversation

@alexslavr

Copy link
Copy Markdown
Contributor

Follow-up for #33510
devextreme-dist internal counterpart should be renamed by adding -internal suffix, otherwise it overwrites original dist package

@alexslavr alexslavr requested a review from a team June 1, 2026 10:39
@alexslavr alexslavr self-assigned this Jun 1, 2026
@alexslavr alexslavr added the 26_1 label Jun 1, 2026
Copilot AI review requested due to automatic review settings June 1, 2026 10:39
@alexslavr alexslavr changed the title CI: Apply set transformation to devextreme-dist-internal package CI: Apply setName transformation to devextreme-dist-internal package Jun 1, 2026
@alexslavr alexslavr changed the title CI: Apply setName transformation to devextreme-dist-internal package CI: Apply setName transformation to devextreme-dist-internal package.json Jun 1, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Ensures the internal build of the devextreme-dist npm artifact does not overwrite the public devextreme-dist package by explicitly renaming the internal package during prepare-package-json.

Changes:

  • Adds setName: "devextreme-dist-internal" to the build:npm:dist:package-json target’s internal configuration.
  • Keeps output separation by continuing to use a distinct distDirectory for the internal artifact.

@alexslavr alexslavr merged commit 5f96402 into 26_1 Jun 1, 2026
103 of 104 checks passed
@alexslavr alexslavr deleted the lavrov/fix-devextreme-dist-build-26_1 branch June 1, 2026 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants