-
Notifications
You must be signed in to change notification settings - Fork 0
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
ESM build #19
ESM build #19
Conversation
cdb2c90
to
41a2bf9
Compare
Ensure the output file is included in the package.
41a2bf9
to
9f03b27
Compare
@papandreou the last commit is not intended to land - it was only necessary to support using a github URL in the PR to Unexpected core to demonstrate it working. Will remove before landing :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks fine! I'm curious why it's necessary to have an esm build of this plugin, though?
@papandreou ah, it’s because the core test suite loads this plugin. In order to minimise complexity it was easier to have an esm build of this and do only the tests in core than much more involved fiddling with the core Makefile to somehow squeeze building it in there. |
Ah, okay. It's fine, really :) |
9f03b27
to
0cfb8d4
Compare
No description provided.