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

Restore support for Node.js < v12 --experimental-modules mode #11

Merged

Conversation

jaydenseric
Copy link
Collaborator

This drops support for Node.js v12 --experimental-modules mode, as discussed here.

A few tooling related improvements and fixes are included; see the changelog entry.

This drops support for Node.js v12 --experimental-modules mode.
I incorrectly assumed the top entry was the most recent.
@mike-marcacci
Copy link
Owner

This looks great; I'm going to (incorrectly) publish this as a non-major change, since I want to revert the breakage I inadvertently caused in a "patch".

@mike-marcacci mike-marcacci merged commit 4d58a4e into mike-marcacci:master May 6, 2019
@jaydenseric
Copy link
Collaborator Author

Perhaps the change log entry “Major” content could be moved under “Patch” 🤔

@mike-marcacci
Copy link
Owner

mike-marcacci commented May 6, 2019

Yes; I'm just rolling those entries up under a single version heading for now, and will invest in clearer headings in the next release. Thanks so much for cleaning up my mess here! I only use native es modules in a couple places, so keeping compatibility there isn't perfectly intuitive for me.

@jaydenseric jaydenseric deleted the experimental-modules-fix branch May 6, 2019 07:19
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