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

Pin solidity version for core contracts to 0.8.15 #135

Closed
wants to merge 3 commits into from
Closed

Conversation

bingen
Copy link
Contributor

@bingen bingen commented Jul 24, 2022

Fixes #126

@bingen
Copy link
Contributor Author

bingen commented Jul 26, 2022

0.8.15 seems too recent. For instance, Trail of Bits recommends up to 0.8.7.
Closing for now, we can re-open and amend if we decide on a particular version.

@bingen bingen closed this Jul 26, 2022
@bingen
Copy link
Contributor Author

bingen commented Jul 27, 2022

A related comment. They are supposed to update the list of recommended versions for 0.8:
crytic/slither#1251 (comment)

@danielattilasimon danielattilasimon deleted the dedaub_a5 branch September 26, 2022 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dedaub A5
1 participant