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

hevm: nicer error message when trying to deploy unlinked bytecode #816

Merged
merged 1 commit into from
Oct 7, 2021

Conversation

d-xo
Copy link
Contributor

@d-xo d-xo commented Sep 23, 2021

Description

Before:

image

After:

image

ref: #802 (comment)

Could perhaps be made even nicer by printing the exact contract with the issue, but we don't have that context available in toCode, and adding it seemed annoying, so I left it like this for now.

Checklist

  • tested locally
  • added automated tests
  • updated the docs
  • updated the changelog

@d-xo d-xo merged commit 1729e1d into master Oct 7, 2021
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

1 participant