Update to lodash 4

[jcope2013]

Relevant change include

• Removed category names from module paths
• contains alias removed in favor of includes
• pluck removed in favor of map
• where removed in favor of filter
• omit no longer takes function argument, use omitBy/isNull
• uniq no longer takes function argument, use uniqBy

https://github.com/lodash/lodash/wiki/Changelog

[stefanpenner]

Awesome thanks!

@homu r+

[homu]

📌 Commit d6b7db4 has been approved by stefanpenner

[homu]

⌛ Testing commit d6b7db4 with merge 3ab5d1f...

[homu]

💔 Test failed - status

[jcope2013]

tests should pass now

[kellyselden]

@homu retry

[stefanpenner]

@homu retry

[stefanpenner]

@home r+

[kellyselden]

@stefanpenner homu doesn't like us anymore apparently

[stefanpenner]

@homu +r (i did typo above, lets try again)

[homu]

⌛ Trying commit ad42f25 with merge d775f9f...

[homu]

☀️ Test successful - status

[trabus]

@jcope2013 I think you may have missed this line:
https://github.com/ember-cli/ember-cli/blob/master/lib/models/file-info.js#L18

If you could move that require up so it's not used inline, that would be awesome.

[trabus]

Looks like AppVeyor is still having issues with the blueprint tests, which are most likely related to the file-info use of lodash/template. Uncertain if there are any changes that would affect the template rendering on Windows, but that seems like a good place to start looking.

[jcope2013]

@trabus not sure they are related to this PR as they are failing in other PR's also #5374 and a whole bunch of other ones

[trabus]

@jcope2013 @kellyselden Looks like they started with this PR:
#5345

It seems the change of the eol character changed the result to something other than what the tests were expecting.

[stefanpenner]

they are failing, (to do timeouts, yes we must address that) but as @trabus pointed out there are also template specific issues in this test run.