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

Feature/0.8 diamond version #222

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

unibeck
Copy link

@unibeck unibeck commented Nov 11, 2021

Closes #160

I copied the latest contracts from here https://github.com/mudgen/diamond-3-hardhat, which use solidity version 0.8. I also had to update the hardhat-deploy specific Diamantaire contract. Lastly, I updated hardhat to compile the contracts with the latest supported version 0.8.9, instead of 0.7.

@unibeck
Copy link
Author

unibeck commented Nov 28, 2021

@mudgen as discussed, it would great to get your stamp of approval, at least on the Diamond specific work. It is pretty straight forward, I copied the latest contract from https://github.com/mudgen/diamond-3-hardhat to solc_0.8/diamond/ and compiled it with solidity v0.8.9. There were some minor changes that I had to make to support the hardhat-deploy specific Diamantaire, so keep and eye out for that.

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.

Solidity 0.8 Diamond Version
1 participant