Skip to content

fix: enable dynamicImport #21

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

Merged
merged 8 commits into from
Jul 15, 2020
Merged

fix: enable dynamicImport #21

merged 8 commits into from
Jul 15, 2020

Conversation

egoist
Copy link
Owner

@egoist egoist commented Jul 15, 2020

because we need to detect dynamic imports

@KagamiChan
Copy link
Contributor

I tried the v2 version and think we're missing a lot of plugins to support a wide range of code, for example, my code requires classProperties

maybe we need to have a way to exclude only flow/flow-commeents

@egoist
Copy link
Owner Author

egoist commented Jul 15, 2020

@KagamiChan there's an option https://github.com/egoist/konan#parse so you can add custom plugins, dynamicImport is required here because we need to detect dynamic imports by default.

@egoist egoist merged commit 39bb586 into master Jul 15, 2020
@egoist egoist deleted the egoist-patch-1 branch July 15, 2020 14:44
@github-actions
Copy link

🎉 This PR is included in version 2.1.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@@ -11,27 +11,18 @@
"index.js"
],
"scripts": {
"test": "jest && npm run lint",
"lint": "xo index.js __test__/*.test.js"
"test": "node test/test.js"
Copy link

Choose a reason for hiding this comment

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

Any particular reason you chose this over uvu test -i fixture ?

Or uvu -r esm test -i fixture for that matter (since esm is already here)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants