Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump to lodash v4. #11

Merged
merged 1 commit into from
May 25, 2016
Merged

Bump to lodash v4. #11

merged 1 commit into from
May 25, 2016

Conversation

samhashemi
Copy link
Contributor

Several lodash functions were renamed in v4:
indexBy -> keyBy
invoke -> invokeBy
rest -> tail
To preserve backwards compability, I kept both the old and new naming.
There is probably a larger clean-up to be done here, but this
is a good first draft to get us to v4.

Several lodash functions were renamed in v4:
`indexBy` -> `keyBy`
`invoke` -> `invokeBy`
`rest` -> `tail`
To preserve backwards compability, I kept both the old and new naming.
There is probably a larger clean-up to be done here, but this
is a good first draft to get us to v4.
@pgilad
Copy link
Member

pgilad commented May 8, 2016

I think tests need to be updated as well, see #9 as well

@lukekarrys
Copy link
Contributor

I'm going to merge this and update the tests.

@lukekarrys lukekarrys merged commit 25ef9ad into AmpersandJS:master May 25, 2016
This was referenced May 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants