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

Upgrade dependencies January 2019 - Related to #2500 #2732

Merged
merged 17 commits into from Jan 16, 2019

Conversation

diego-G
Copy link

@diego-G diego-G commented Jan 14, 2019

Review checklist

@MaciejBaj
Copy link
Contributor

MaciejBaj commented Jan 14, 2019

Taking into account how massive the code changes are - 60k changed line, we should suspend accepting the PR until the DB feature (https://github.com/LiskHQ/lisk/tree/feature/extensible-data-persistence-model) branch gets merged into development.

@diego-G
Copy link
Author

diego-G commented Jan 14, 2019

@MaciejBaj I understand your concerns. I can remove prettier commit which is introducing most of the changes, reducing that way the PR size significantly. What do you think?

@diego-G diego-G force-pushed the 2500-update_dependencies_january_2019 branch from 9c86a4e to 499e577 Compare January 14, 2019 14:32
MaciejBaj
MaciejBaj previously approved these changes Jan 14, 2019
@MaciejBaj MaciejBaj merged commit 1d60ebf into development Jan 16, 2019
@MaciejBaj MaciejBaj deleted the 2500-update_dependencies_january_2019 branch January 16, 2019 10:27
lsilvs pushed a commit that referenced this pull request Jan 30, 2019
lsilvs added a commit that referenced this pull request Jan 30, 2019
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.

None yet

2 participants