-
Notifications
You must be signed in to change notification settings - Fork 243
'lodash.assign' vs 'Object.assign' #28
Comments
If we do that, we'd have to drop support for Node < 4 which is fine, we'd just need to push a breaking change. Feel free to file a PR. |
Oh, right, |
Looks like this is breaking in electron. Ref: https://gist.github.com/DavidWells/2c79ce5864f2b3c224423c7db53b3b17 https://github.com/chentsulin/electron-react-boilerplate/ throws when trying to run the Should we revert this back to lodash for now? =) |
Sorry, |
Thanks Kent! Take the day off for your birthday. You deserve it. I'll figure this out :)
|
For how it it used, wouldn't
Object.assign
work overlodash.assign
? I tried swapping them out and the tests all passed.dist
was .22kb lighter, too.If I'm not missing something here, I can submit a Pull Request.
The text was updated successfully, but these errors were encountered: