Skip to content

ESM fix #12

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

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

ESM fix #12

wants to merge 2 commits into from

Conversation

tpluscode
Copy link

Fixes #9

In addition to type: module, the modules must all be referenced with .js extension. As a side note, I can highly recommend using eslint-plugin-require-extensions which ensures the extensions are used

The second commit is a little unrelated but it's the minimum I had to do to have the compilation work. Since you do not lock on a specific version of dependencies, I got type errors from @types/babel__traverse. Most likely caused by an old version of typescript, hence I added v5 to the dev dependencies. Let me know if you'd like this handled differently

@TomasCarlson
Copy link

@andreasbm would appreciate this merge :)

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.

@appnest/masonry-layout doesn't appear to be written in CJS, but also doesn't appear to be a valid ES module
2 participants