Move devextreme-dist to packages and add documentation files#33240
Merged
Move devextreme-dist to packages and add documentation files#33240
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR relocates the devextreme-dist workspace definition from a build-artifact folder into packages/ and ensures the published devextreme-dist output includes basic documentation files.
Changes:
- Removes
packages/devextreme/artifacts/npm/devextreme-distfrom pnpm workspaces and registerspackages/devextreme-distas the workspace package. - Updates DevExtreme npm build pipeline to source
devextreme-distmetadata from the new workspace package and to copyREADME.md/LICENSE.mdinto the publish output. - Adds
README.mdandLICENSE.mdfordevextreme-distand updates itspackage.jsonpublish config.
Reviewed changes
Copilot reviewed 7 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-workspace.yaml | Stops treating the artifacts-based devextreme-dist folder as a workspace package. |
| pnpm-lock.yaml | Updates workspace importer entries to reflect packages/devextreme-dist and its publish directory. |
| packages/devextreme/gulpfile.js | Adjusts clean exclusions related to devextreme-dist artifacts. |
| packages/devextreme/build/gulp/npm.js | Builds devextreme-dist package metadata from the new workspace and copies README/LICENSE to the dist output. |
| packages/devextreme/.gitignore | Removes the exception for tracking an artifacts-based devextreme-dist/package.json. |
| packages/devextreme-dist/README.md | Adds package documentation for devextreme-dist. |
| packages/devextreme-dist/LICENSE.md | Adds licensing reference document for devextreme-dist. |
| packages/devextreme-dist/package.json | Adds publishConfig and minor formatting cleanup. |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
Comments suppressed due to low confidence (1)
packages/devextreme-dist/package.json:57
packages/devextreme-dist/package.jsonis missing alicensefield. Other published packages in this repo set it (e.g.,packages/devextreme/package.jsonuses"license": "SEE LICENSE IN README.md"). Since this package now ships its ownLICENSE.md, consider adding an explicitlicensevalue (orSEE LICENSE IN LICENSE.md) to avoid npm metadata warnings and make licensing discoverable.
IlyaKhD
approved these changes
Apr 14, 2026
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
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.
Cherry-pick of #33150 & #33232