-
Notifications
You must be signed in to change notification settings - Fork 14
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
Update to ES modules #38
Comments
Runs into problem Could disable that rule, and rely on CI unit tests to catch e.g. Node 14 issues. But currently that doesn't cover the |
Disable the rule, and use es-compat itself to check compatibility of its own source code? (!) But that would be disconnected from |
Need to switch to https://github.com/eslint-community/eslint-plugin-n because the author (same one as eslint-plugin-es) has also abandoned his node plugin |
Blocked on Jest ESM support - https://jestjs.io/docs/ecmascript-modules |
to do as part of this eventually:
|
Also see this about importing browser compat data https://github.com/mdn/browser-compat-data/releases/tag/v5.0.0 |
Update the code of the plugin itself to use
import
andexport
rather thanrequire
andmodule.exports
.Update the lint rules in eslintrc to require this style, and prohibit CommonJS style.
The text was updated successfully, but these errors were encountered: