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

Migrate to ESM #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Migrate to ESM #15

wants to merge 2 commits into from

Conversation

ngzhian
Copy link

@ngzhian ngzhian commented Aug 17, 2020

We've recently migrated Chrome DevTools to ESM, and have traditionally
been using acorn as the parser for various features including the
prettifier. In the context of https://crbug.com/1084349 we are looking
into including private fields support in Chrome DevTools via this
plugin, and in order to make that easier with our build system, I've
added support for ESM output here in addition to the CJS output.

Similar change to
acornjs/acorn-private-methods#4.

We've recently migrated Chrome DevTools to ESM, and have traditionally
been using acorn as the parser for various features including the
prettifier. In the context of https://crbug.com/1084349 we are looking
into including private fields support in Chrome DevTools via this
plugin, and in order to make that easier with our build system, I've
added support for ESM output here in addition to the CJS output.

Similar change to
acornjs/acorn-private-methods#4.
@ngzhian
Copy link
Author

ngzhian commented Sep 22, 2020

Hi, does this PR look okay?

index.js Outdated Show resolved Hide resolved
Co-authored-by: Adrian Heine né Lang <[email protected]>
@ngzhian
Copy link
Author

ngzhian commented Dec 8, 2020

Ping

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