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

Update/node-modules html-loader, json-loader, imports-loader #1230

Merged
merged 3 commits into from
Dec 18, 2015

Conversation

artpi
Copy link
Contributor

@artpi artpi commented Dec 3, 2015

Testing

make run
There shouldnt be any build errors

@lancewillett lancewillett added Build [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Dec 4, 2015
@blowery
Copy link
Contributor

blowery commented Dec 17, 2015

do you happen to know if we use the html-loader for anything?

@rralian
Copy link
Contributor

rralian commented Dec 17, 2015

do you happen to know if we use the html-loader for anything?

Not much, but for fun I tried webpack without it and it choked on a file inside the component-tip component we use for tooltips. So it's at least necessary for that.

ERROR in ./~/component-tip/template.html
Module parse failed: /Users/robertralian/Projects/wp-calypso/node_modules/component-tip/template.html Line 1: Unexpected token <
You may need an appropriate loader to handle this file type.
| <div class="tip tip-hide">
|   <div class="tip-arrow"></div>
|   <div class="tip-inner"></div>
 @ ./~/component-tip/index.js 14:18-44
webpack: bundle is now VALID.

@rralian
Copy link
Contributor

rralian commented Dec 17, 2015

They're not much for changelogs are they? I did some sanity testing running in different environments and am seeing no problems. 👍

@rralian rralian added [Status] Ready to Merge and removed [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. labels Dec 17, 2015
artpi added a commit that referenced this pull request Dec 18, 2015
Update/node-modules html-loader, json-loader, imports-loader
@artpi artpi merged commit bf459cd into master Dec 18, 2015
@artpi artpi deleted the update/node_modules__loaders branch December 18, 2015 09:24
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.

5 participants