Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

Update ethereumjs-util #24

Merged
merged 2 commits into from
Nov 13, 2017
Merged

Update ethereumjs-util #24

merged 2 commits into from
Nov 13, 2017

Conversation

fanatid
Copy link
Contributor

@fanatid fanatid commented Nov 6, 2017

Please publish with update :)

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.234% when pulling 0d8e219 on update-ethereumjs-util into f4a7f4e on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 87.234% when pulling aca3a46 on update-ethereumjs-util into f4a7f4e on master.

@holgerd77
Copy link
Member

@wanderer @kumavis @axic If no one of you guys protest within a day or two I would merge this, not seeing a reason why not since tests are all passing and usage of ethereumjs-util is not very broad in scope.

jwasinger
jwasinger approved these changes Nov 13, 2017
View reviewed changes
Copy link
Contributor

@jwasinger jwasinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jwasinger jwasinger merged commit 653beca into master Nov 13, 2017
@fanatid fanatid deleted the update-ethereumjs-util branch November 13, 2017 21:21
@fanatid
Copy link
Contributor Author

fanatid commented Nov 14, 2017

@kumavis @wanderer can you publish new version?
I want update dependencies and exclude ethereumjs-util@4 in project.
Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@jwasinger jwasinger jwasinger approved these changes

Assignees

@wanderer wanderer

@kumavis kumavis

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@fanatid @coveralls @holgerd77 @jwasinger @wanderer @kumavis