Skip to content
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

feat: add buildin loaders for simplifying usage #4

Closed
wants to merge 1 commit into from

Conversation

AdvancedCat
Copy link
Contributor

I noticed you add a feature about using react/typescript loader to transform code. Nice feature.
Maybe could let loader more simplifier by removing babel or loaders script.
like this:

<!--
<script src="https://unpkg.com/@babel/standalone/babel.min.js"></script>
<script src="https://unpkg.com/inline-module/dist/loaders.js"></script>
-->
<script src="https://unpkg.com/inline-module/dist/core.js" setup></script>

core.js can scan type="inline-script" script automatically and can check whether import babel/react/typescript loader or not.

I pull a request to review, it's counting on you.

@akira-cn
Copy link
Contributor

akira-cn commented Mar 7, 2022

The idea is pretty well but users may prefer other CDN files such as https://cdnjs.com/libraries/babel-standalone. I think we should give more choices to users.

@akira-cn akira-cn closed this Mar 7, 2022
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.

None yet

2 participants