Skip to content

Feature/remove systemjs from demos. Сleanup - #34836

Merged
vorobey merged 20 commits into
DevExpress:feature/remove-systemjs-from-demosfrom
GoodDayForSurf:feature/remove-systemjs-from-demos-clean
Aug 20, 2026
Merged

Feature/remove systemjs from demos. Сleanup#34836
vorobey merged 20 commits into
DevExpress:feature/remove-systemjs-from-demosfrom
GoodDayForSurf:feature/remove-systemjs-from-demos-clean

Conversation

@GoodDayForSurf

@GoodDayForSurf GoodDayForSurf commented Aug 19, 2026

Copy link
Copy Markdown
Contributor
  • Removed unused Rollup-built demo extras (openai tarball / create-tgz-packages) and the Rollup plugins that produced them.
  • Dropped the prepare-js script and the CI steps that only existed to generate those tgz bundles.
  • Angular bundler resolves DevExtreme from ESM (transpiled-esm-npm/esm), not CJS.
  • ESLint: dropped ignores for shared/loaders and config.js, the System global, and the unused configs/** block.
  • modulePrefix and window.config?.packageConfigPaths removed from all Angular demos.
  • Localization switched from npm:…json!json to normal JSON imports; systemjs-imports.d.ts files deleted.
  • SystemJS quirks (npm:, !json) removed from bundlers; only globalize / anti-forgery aliases remain.
  • Dead entries removed from .gitignore: shared/empty-file.js and Demos/**/config.js.
  • Demos README rewritten for the current flow: demos:prepare / demos:start, on-demand bundles, no SystemJS.

vorobey and others added 16 commits August 10, 2026 11:12
Co-authored-by: Arman Jivanyan <arman.jivanyan@devexpress.com>
Co-authored-by: Arman Jivanyan <arman.jivanyan@devexpress.com>
Co-authored-by: Arman Jivanyan <arman.jivanyan@devexpress.com>
Co-authored-by: Arman Jivanyan <arman.jivanyan@devexpress.com>
…om/DevExpress/DevExtreme into feature/remove-systemjs-from-demos

# Conflicts:
#	pnpm-lock.yaml
…om/DevExpress/DevExtreme into feature/remove-systemjs-from-demos

# Conflicts:
#	.github/workflows/visual-tests-demos.yml
#	apps/demos/Demos/PivotGrid/StandaloneFieldChooser/Angular/app/app.component.ts
#	apps/demos/project.json
#	apps/demos/utils/server/csp-bundle-angular.js
#	apps/demos/utils/server/csp-bundle.js
#	apps/demos/utils/shell/server.js
…om/DevExpress/DevExtreme into feature/remove-systemjs-from-demos

# Conflicts:
#	.github/workflows/visual-tests-demos.yml
#	apps/demos/.gitignore
#	apps/demos/project.json
#	apps/demos/scripts/prepare-js-configs.js
#	apps/demos/utils/server/csp-bundle-angular.js
#	apps/demos/utils/server/csp-bundle.js
#	apps/demos/utils/shell/server.js
#	pnpm-lock.yaml
@GoodDayForSurf GoodDayForSurf changed the title Feature/remove systemjs from demos clean Feature/remove systemjs from demos. Сleanup Aug 19, 2026
@GoodDayForSurf
GoodDayForSurf requested a lite review from Copilot August 19, 2026 12:08

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

@GoodDayForSurf
GoodDayForSurf marked this pull request as ready for review August 19, 2026 14:36
@GoodDayForSurf
GoodDayForSurf requested review from a team as code owners August 19, 2026 14:36
@GoodDayForSurf
GoodDayForSurf requested review from a team and a lite review from Copilot August 19, 2026 14:36

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Comment thread apps/demos/scripts/prepare-js-configs.js
Comment thread apps/demos/package.json
@mpreyskurantov
mpreyskurantov requested a lite review from Copilot August 19, 2026 15:01

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of files (300). Try reducing the number of changed files and requesting a review from Copilot again.

Comment thread apps/demos/utils/server/tsconfig.csp-bundle-angular.json Outdated
@vorobey
vorobey merged commit b0e6c3e into DevExpress:feature/remove-systemjs-from-demos Aug 20, 2026
155 of 183 checks passed
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.

7 participants