Skip to content

Conversation

@lforst
Copy link

@lforst lforst commented Apr 17, 2023

fixes #211

Rollup takes care of correctly transpiling the import meta stuff

@lforst lforst requested review from AbhiPrasad, Lms24 and mydea April 17, 2023 16:33
Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Damn, ESM/CJS strikes again...

@lforst lforst merged commit 5376687 into main Apr 17, 2023
@lforst lforst deleted the lforst-fix-no-require branch April 17, 2023 16:46
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.

require is not defined in ES module scope, you can use import instead

3 participants