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

Fix issue #34 #42

Closed
wants to merge 2 commits into from
Closed

Fix issue #34 #42

wants to merge 2 commits into from

Conversation

ygnr
Copy link

@ygnr ygnr commented Mar 6, 2017

Returns a string for address type from decodeSingle function.

@coveralls
Copy link

Coverage Status

Coverage decreased (-1.5%) to 81.667% when pulling 98c6c18 on ygnr:master into 4a33f23 on ethereumjs:master.

@danjm
Copy link

danjm commented Jun 3, 2019

Issue #34 was resolved by #57

Note the comment in that #57 (comment) which identifies the same problem as #34 and requests a unit test, which was subsequently added.

So this can be closed

@holgerd77
Copy link
Member

Thanks @danjm, feel free to close yourself next time. Worst thing that can happen is that someone will reopen, and even that outcome will add some value to the thread since there is then some renewed expression interest in the thread. 👀🙂

@holgerd77 holgerd77 closed this Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants