no babel-loader node_modules exclude/web_modules Fixes #1278

[graingert]

and babel@7 so it's safe to do so.

Fixes #1278

also remove ancient unused version of jquery hiding in web_modules.

[jsf-clabot]

All committers have signed the CLA.

[graingert]

I had @nobitagit try this out in IE11 and it works great.

[graingert]

thank you @sindresorhus for dragging us into the future!

[codecov]

Codecov Report

Merging #1279 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1279   +/-   ##
=======================================
  Coverage   75.72%   75.72%           
=======================================
  Files           5        5           
  Lines         482      482           
  Branches      156      156           
=======================================
  Hits          365      365           
  Misses        117      117

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c1ed7a...377b146. Read the comment docs.

[graingert]

(replying from #1273 (comment))

@julianxhokaxhiu are you sure that's not in your own code?

Here's the bundled JS https://unpkg.com/@procensus/[email protected]/client/index.bundle.js

[julianxhokaxhiu]

Yes definitely sure. Feel free to check the project here: https://github.com/julianxhokaxhiu/polysticky.js

//EDIT:
Found also the same issue here: vercel/next.js#2747

It seems that downgrading strip-ansi is the solution: https://github.com/zeit/next.js/pull/2860/commits

[graingert]

@julianxhokaxhiu you've not switched to @procensus/webpack-dev-server in polysticky. Can you publish a branch with your changes?

[julianxhokaxhiu]

I tested it locally, and it didn't work. So no need to push a commit that is useless.

Feel free though to try on your side. Remember that you can try locally by using npm run serve

[yyx990803]

Just to explain why #1273 was needed instead of doing this: the webpack/babel transpilation only affects the bundles used in live mode. When the user uses WDS via the CLI with --inline flag, the client is injected by adding an entry to the users' bundle. So this PR would be an incomplete fix.

[SpaceK33z]

Closing because #1273 fixed the same issue.