-
Notifications
You must be signed in to change notification settings - Fork 55
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
fix: improve cjs esm compatibility #47
base: main
Are you sure you want to change the base?
fix: improve cjs esm compatibility #47
Conversation
Will this fix #44 as well do you think? |
Yes, this PR also resolves that issue. There was another issue now Thank you for making me aware of that, it helped me catch the bug 👍 . |
I also fixed a wrong import in tests and small lint issue that was introduced by me. Now it should be better. |
@developit When you get the time, please check out the PR as it should resolve quite a lot of issues. |
Possibly a fix for #48 too |
The build is broken for 7 months in this PR isn't reviewed after 4 months. Maybe time to fork this 😢 |
Fixes: #46 and also #36 and #44