-
Notifications
You must be signed in to change notification settings - Fork 19
Support solc v0.8.29 #261
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
Support solc v0.8.29 #261
Conversation
Signed-off-by: Cyrill Leutwiler <[email protected]>
Signed-off-by: Cyrill Leutwiler <[email protected]>
Signed-off-by: Cyrill Leutwiler <[email protected]>
Signed-off-by: Cyrill Leutwiler <[email protected]>
Signed-off-by: Cyrill Leutwiler <[email protected]>
Please update the solidity version for wasm compiler. |
Signed-off-by: xermicus <[email protected]>
Good catch, thanks! |
pull_request: | ||
branches: ["main"] | ||
types: [opened, synchronize, labled, unlabled] | ||
types: [opened, synchronize, labeled, unlabeled] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was going mad why this shit wasn't working lol.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah as obscure as it gets, I only caught it because of the VS code extension complained about it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah why isn't it an error to use non existent types :(
Signed-off-by: xermicus <[email protected]>
No observable changes w.r.t. YUL compilation.