Skip to content

Modernize JavaScript build and test infrastructure#2854

Merged
jstone-lucasfilm merged 1 commit intoAcademySoftwareFoundation:mainfrom
jstone-lucasfilm:dev_javascript
Apr 7, 2026
Merged

Modernize JavaScript build and test infrastructure#2854
jstone-lucasfilm merged 1 commit intoAcademySoftwareFoundation:mainfrom
jstone-lucasfilm:dev_javascript

Conversation

@jstone-lucasfilm
Copy link
Copy Markdown
Member

This changelist modernizes the JavaScript build and test infrastructure in MaterialX, including the following changes:

  • Migrate from CommonJS to native ES modules, adding the EXPORT_ES6 flag to the Emscripten build.
  • Replace the Karma test framework with Playwright for browser tests.
  • Adopt npm workspaces, consolidating dependency management under a single root package.
  • Replace third-party build utilities with a native Node.js script for test setup.

This changelist modernizes the JavaScript build and test infrastructure in MaterialX, including the following changes:

- Migrate from CommonJS to native ES modules, adding the EXPORT_ES6 flag to the Emscripten build.
- Replace the Karma test framework with Playwright for browser tests.
- Adopt npm workspaces, consolidating dependency management under a single root package.
- Replace third-party build utilities with a native Node.js script for test setup.
Copy link
Copy Markdown
Contributor

@ashwinbhat ashwinbhat left a comment

Choose a reason for hiding this comment

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

@jstone-lucasfilm this modernization is very helpful, thanks for taking care of this.
I locally tested the PR on a clean setup LGTM.

@jstone-lucasfilm jstone-lucasfilm merged commit 1cd6459 into AcademySoftwareFoundation:main Apr 7, 2026
36 checks passed
@jstone-lucasfilm jstone-lucasfilm deleted the dev_javascript branch April 7, 2026 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants