-
Notifications
You must be signed in to change notification settings - Fork 62
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
feat(docs): update for solc 0.8.23 support #373
feat(docs): update for solc 0.8.23 support #373
Conversation
@danceratopz This is a work in progress. I began by looking at your changes for solc 0.8.22 and replicating them. Making edits to the README and quickstart seemed straightforward. For the others like Regarding |
Thanks a lot for making a start on this!
Regarding these commits (btw, you can link commit hashes in PR comments for convenience π):
The task here is to work out whether we're doing everything we can to disable the optimizations At the moment these are the options we use: execution-spec-tests/src/ethereum_test_tools/code/yul.py Lines 75 to 79 in 8c7cb87
If 0.8.23 makes it easier to disable optimizations, it would be nice to add this. We would only add it to solc for versions >=0.8.23 obviously though.
If you're interested, feel free to dig in. If not, zero worries, we can add this to a separate issue or I can have a look π. |
I've pushed updates. I'd like to dig into the yul optimization task but as a separate task. From what you've stated the solution seems like an if statement that checks if the version is greater than or equal to 0.8.23 and adds |
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.
Thanks, looks good to me! I just fixed up black and added a changelog entry.
Also thanks for adding #395.
ποΈ Description
Updated framework tests and docs for solc 0.8.23
π Related Issues
#345
β Checklist
mkdocs serve
locally and verified the auto-generated docs for new tests in the Test Case Reference are correctly formatted.