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

assert message strings #2449

Closed
benjiqq opened this issue Sep 3, 2021 · 0 comments · Fixed by #2509
Closed

assert message strings #2449

benjiqq opened this issue Sep 3, 2021 · 0 comments · Fixed by #2509
Labels
bug Bug that shouldn't change language semantics when fixed. Easy Pickings Used to denote issues that should be easy to implement help wanted

Comments

@benjiqq
Copy link
Contributor

benjiqq commented Sep 3, 2021

Version Information

latest vyper

What's your issue about?

when I do

assert 1==1, concat("a","b")

I get:
vyper.exceptions.InvalidType: Reason must UNREACHABLE or a string literal
@benjiqq benjiqq mentioned this issue Sep 3, 2021
Closed
@charles-cooper charles-cooper added bug Bug that shouldn't change language semantics when fixed. Easy Pickings Used to denote issues that should be easy to implement help wanted labels Oct 20, 2021
@charles-cooper charles-cooper mentioned this issue Oct 23, 2021
Merged
@emc415 emc415 mentioned this issue Dec 19, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Bug that shouldn't change language semantics when fixed. Easy Pickings Used to denote issues that should be easy to implement help wanted
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

allow arbitrary revert reason strings charles-cooper/vyper
2 participants
@charles-cooper @benjiqq