Skip to content
This repository has been archived by the owner on Mar 20, 2022. It is now read-only.

Use Lodash v4 and shortened paths #52

Merged
merged 1 commit into from
Jan 13, 2016
Merged

Use Lodash v4 and shortened paths #52

merged 1 commit into from
Jan 13, 2016

Conversation

wbinnssmith
Copy link
Contributor

Pros:

  • New! Shiny! Hot off the presses lodash v4.0.0
  • Likely longer maintained
  • Shorter require paths

Cons:

  • Bundled size goes from ~4.3k min+gz to 5.6kgz. ¯_(ツ)_/¯
  • Drops support for IE < 9, like upcoming React v0.15

Tests continue to pass. Thanks Dan!

@gaearon
Copy link
Contributor

gaearon commented Jan 13, 2016

Thanks! I'll leave this hanging for some time until v4 stabilizes and then merge.

@gaearon gaearon merged commit 4438438 into paularmstrong:master Jan 13, 2016
@gaearon
Copy link
Contributor

gaearon commented Jan 13, 2016

Gah, I decided to give it a go right away :D

@gaearon
Copy link
Contributor

gaearon commented Jan 13, 2016

Out in 2.0.0.

@wbinnssmith
Copy link
Contributor Author

🎉 🙌

@lock
Copy link

lock bot commented May 7, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot added the Outdated label May 7, 2018
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
Outdated
Projects
None yet
Milestone
No milestone
2 participants
@wbinnssmith @gaearon