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.

Sign up for GitHub

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

Jump to bottom

chore(lib): update lodash and redux dependencies #209

Merged
merged 1 commit into from
Oct 12, 2018
Merged

chore(lib): update lodash and redux dependencies #209

merged 1 commit into from
Oct 12, 2018

Conversation

AntJanus
Copy link
Collaborator

@AntJanus AntJanus commented Oct 12, 2018 •
edited
Loading

Closes #203

passes tests.

This PR does a couple of things:

  1. allows Redux 4 to be a peer dependency
  2. updated lodash to newest version to bypass a security warning
  3. unified lodash loading functions to use the all-encompassing lodash package; however, it only pulls in and compiles with the functions ng-redux uses

@AntJanus AntJanus requested a review from deini October 12, 2018 21:07
@AntJanus AntJanus mentioned this pull request Oct 12, 2018
Closed
7 tasks
@AntJanus AntJanus merged commit 8da68d6 into master Oct 12, 2018
@AntJanus AntJanus deleted the chore/update-lodash branch October 12, 2018 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@deini deini Awaiting requested review from deini

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AntJanus