Skip to content

ES module support on Node 16 #46

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

Closed
wants to merge 7 commits into from
Closed

ES module support on Node 16 #46

wants to merge 7 commits into from

Conversation

benjay10
Copy link

@benjay10 benjay10 commented Nov 3, 2022

Add support for the installation of ES modules in a project and base on Node 16.

Since Node 14 is reaching end of life, many packages are dropping support for for it. Many packages also prefer/require the use of ES modules.

I'm not an expert, so this PR is based on the changes from the two branches that introduce ES module support and the Node 16 support. After testing it looks like the build process does not put the package.json file in the mu package. This is a requirement for any Node package, especially when Node looks for settings like "type": "module" in the package. This PR tries to fix that by just copying that file while also copying the Babel config for transpilation, but the transpile-sources.sh is convoluted, so this needs to be tested/rewritten more thoroughly.

elpoelma and others added 7 commits March 31, 2022 16:09
When the app is being compiled as a module, so should also the mu
package. Also included the package.json files in the package's final
product because Node uses that to run the project(?).
I haven't tested the changes to the TypeScript configuration.
@MikiDi
Copy link
Contributor

MikiDi commented Feb 15, 2023

The Node-upgrade part of this is superseded by #51

@MikiDi MikiDi mentioned this pull request Jul 10, 2023
@benjay10 benjay10 closed this Aug 21, 2023
@benjay10 benjay10 deleted the feature/es-module-support-on-16 branch August 21, 2023 12:25
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.

3 participants