ember-cli / ember-cli / 8111 / 2
94%
master: 94%

DEFAULT BRANCH: master
Ran
Files 129
Run time 8s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 92.881%. Remained the same
8111.2

push

travis-ci

homu 
Auto merge of #5036 - patocallaghan:native-dependencies, r=chadhietala

Remove native bundled dependencies

Re: https://github.com/ember-cli/ember-cli/issues/5028#issuecomment-153405634

@stefanpenner this is more a placeholder for discussion rather than a finished PR. Would appreciate you chiming in with your thoughts whether this is even on the right track or not.

## ws.js
It seems the issue of failing native dependencies has been fixed in npm 3.3.12

* Optional dependencies causing an install to fail was fixed in https://github.com/npm/npm/commit/2289234e1 See https://github.com/npm/npm/issues/9643 for the related issue
* running `--no-optional` should not try to install bundled dependencies in deps of deps

This means we don't need the patch for `ws.js` as `bufferutil` and `utf8-validation` will not be installed.

## patch-engine-io-client.js
it appears the reference to the `tgz` [has been removed](https://github.com/socketio/engine.io-client/pull/401/files#diff-b9cfc7f2cdf78a7f4b91a753d10865a2L38) but it hasn't shipped in a release yet . Not entirely sure what we can do here though besides pushing for a release or making our own fork.

### BUT

there's a bug with the 3.3.12 version of NPM which prevents `./bin/prepare-release` from finishing. If you try to `npm uninstall` or `npm unlink` a `npm link`'d global module, the build fails. This means `prepare-release` fails at the second last step when uninstalling the global `ember-cli`.  See here for more on the issue https://github.com/npm/npm/issues/9980.

We could possibly do a `which ember`, work out where the global ember is installed, and manually `rm -rf` it?

3927 of 4228 relevant lines covered (92.88%)

5677.77 hits per line

Source Files on job 8111.2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line