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

Bugfix/issue 84 - Support to solidity 0.8.0 #85

Closed
wants to merge 1 commit into from
Closed

Bugfix/issue 84 - Support to solidity 0.8.0 #85

wants to merge 1 commit into from

Conversation

joaquin-alfaro
Copy link

What does this PR do?

Fixes issue when generating java classes using pragma 0.8.0

Where should the reviewer start?

from the opened issue #84

Why is it needed?

It is needed to generate the code of smart contracts using pragma 0.8.0 or superior.

…idity using the option "--combined-json" is a JSON field instead a String.

This update adds support to this behavior.
@tmac2yt
Copy link

tmac2yt commented Jan 30, 2022

Hi,I'm facing the same issue with you.Can you give me a plugin? @josh-richardson

@tmac2yt
Copy link

tmac2yt commented Jan 30, 2022

Hi,when will the offical update the latest version on Maven repository that fix this issue. @xaviarias

@gsmorag
Copy link

gsmorag commented Feb 22, 2022

When is it going to be released?

@gtebrean
Copy link
Contributor

Thanks for fix! I added to #89 and will release soon.

@andrii-kl
Copy link
Contributor

#89
the fix + tests

@andrii-kl andrii-kl closed this Feb 23, 2022
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

6 participants