-
Notifications
You must be signed in to change notification settings - Fork 22
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
Add ESM support #104
base: main
Are you sure you want to change the base?
Add ESM support #104
Conversation
…ts into feat/mjs-support � Conflicts: � package.json
@calebdwilliams, I wonder what would be better: to use |
Codecov Report
@@ Coverage Diff @@
## main #104 +/- ##
=======================================
Coverage 92.06% 92.06%
=======================================
Files 5 5
Lines 189 189
Branches 32 32
=======================================
Hits 174 174
Misses 4 4
Partials 11 11
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@Lodin in my experience exports is spotty until everything adopts [email protected]. I’d definitely mark that as a to-do but maybe not right away unless it’s absolutely needed. |
Also @Lodin let me know when this is final and I’ll try it out locally. |
Resolves #93