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

chore: use @babel/register instead of esm #16

Merged
merged 3 commits into from
Nov 27, 2022
Merged

Conversation

ota-meshi
Copy link
Member

@ota-meshi ota-meshi commented Nov 27, 2022

This PR is a PR to the eslint-8 branch.
This PR uses esbuild-register @babel/register instead of the esm package. The esm package doesn't seem to work when mixed with real ESM modules.

mochajs/mocha#4671

@codecov-commenter
Copy link

codecov-commenter commented Nov 27, 2022

Codecov Report

❗ No coverage uploaded for pull request base (eslint-8@f01cb20). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             eslint-8      #16   +/-   ##
===========================================
  Coverage            ?   96.01%           
===========================================
  Files               ?       13           
  Lines               ?      653           
  Branches            ?        0           
===========================================
  Hits                ?      627           
  Misses              ?       26           
  Partials            ?        0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@ota-meshi
Copy link
Member Author

Hmm. It doesn't seem to work. It seems to hang on Node v12.

@ota-meshi ota-meshi changed the title chore: use esbuild-register instead of esm. chore: use @babel/register instead of esm. Nov 27, 2022
@ota-meshi ota-meshi changed the title chore: use @babel/register instead of esm. chore: use @babel/register instead of esm Nov 27, 2022
@ota-meshi
Copy link
Member Author

I changed to use @babel/register instead.
That seems to work, but changing the extension from *.js to *.mjs without using register might also work.

@MichaelDeBoey MichaelDeBoey merged commit c903823 into eslint-8 Nov 27, 2022
@MichaelDeBoey MichaelDeBoey deleted the eslint-8-patch branch November 27, 2022 15:28
MichaelDeBoey pushed a commit that referenced this pull request Nov 27, 2022
* chore: use esbuild-register instead of esm.

* chore: add esbuild to deps

* chore: use @babel/register instead of esbuild-register
MichaelDeBoey added a commit that referenced this pull request Nov 27, 2022
* feat: support ESLint 8.x

* chore: use `@babel/register` instead of `esm` (#16)

* chore: use esbuild-register instead of esm.

* chore: add esbuild to deps

* chore: use @babel/register instead of esbuild-register

Co-authored-by: Yosuke Ota <otameshiyo23@gmail.com>
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