Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build ESM and CommonJS modules #1858

Merged
merged 3 commits into from
Dec 22, 2021
Merged

Conversation

sdirix
Copy link
Member

@sdirix sdirix commented Dec 22, 2021

Use Rollup to bundle ESM and CommonJS modules for all non-Angular
packages. Angular builds are done via ngc.

ESM bundles target 'esnext' while CommonJS bundles continue to
target ES5.

Also adjusts Vue dependencies so that Vue and the respective
template compiler are of the same version.

Also removes the React Material extended renderer set

Copy link
Member

@AlexandraBuzila AlexandraBuzila left a comment

Choose a reason for hiding this comment

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

I did not test the changes, but the code looks good to me!

Use Rollup to bundle ESM and CommonJS modules for all non-Angular
packages. Angular builds are done via ngc.

ESM bundles target 'esnext' while CommonJS bundles continue to
target ES5.

Also adjusts Vue dependencies so that Vue and the respective
template compiler are of the same version.
As '@mui/lab' is now a core dependency we no longer need to
maintain two different renderer sets. The autocomplete
renderers are now part of the normal renderer set.
@coveralls
Copy link

coveralls commented Dec 22, 2021

Coverage Status

Coverage decreased (-1.1%) to 84.242% when pulling 1fb9666 on sdirix:rollup-esm into 81f1f19 on eclipsesource:master.

Updates nyc and IstanbulJS to make the CI code coverage
measurement less error prone.

Also increase Node version in CI builds to 14 and skip
tests on MacOS for now as the cross environment
variables don't seem to properly work there.
@sdirix sdirix merged commit 901b2a6 into eclipsesource:master Dec 22, 2021
@sdirix sdirix deleted the rollup-esm branch December 22, 2021 20:35
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.

Remove extended renderer set in React Material Switch module system
3 participants