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.

Sign up for GitHub

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

Jump to bottom

remove babel-loader #47

Closed
Eoksni opened this issue Jun 6, 2017 · 1 comment
Closed

remove babel-loader #47

Eoksni opened this issue Jun 6, 2017 · 1 comment
Assignees
@Eoksni
Labels
refactor
Milestone
v0.1.0

Comments

@Eoksni
Copy link
Member

Eoksni commented Jun 6, 2017

No description provided.

@Eoksni Eoksni added the refactor label Jun 6, 2017
@Eoksni Eoksni added this to the v0.1.0 milestone Jun 6, 2017
@Eoksni Eoksni self-assigned this Jun 6, 2017
Eoksni added a commit that referenced this issue Jun 8, 2017
@Eoksni
closed #47 - remove babel-loader
bc2e52a
@Eoksni
Copy link
Member Author

Eoksni commented Jun 8, 2017

I need to have it so that inject-loader can work properly (because import statements are needed to be converted to require ones). It also seems that using babel for that is a recommended approach as in plasticine/inject-loader#27.

So I kept babel-loader but used it only for modules to which inject-loader is applied.

@Eoksni Eoksni closed this as completed Jun 8, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Eoksni Eoksni

Labels
refactor
Projects
None yet
Milestone
v0.1.0
Development

No branches or pull requests
1 participant
@Eoksni