-
Notifications
You must be signed in to change notification settings - Fork 189
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: bump solidity to 0.8.21 #1473
Conversation
🦋 Changeset detectedLatest commit: 5817aab The changes in this PR will be included in the next version bump. This PR includes changesets to release 29 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
tried moving some events around as noted in #1448 but got weird natspec errors
|
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.
looks good! let's add a changeset, i think this might be a breaking change as it requires consumers to update their solc version too
c82373f
to
9367fa4
Compare
I am now kinda wondering if we want to leave the pragma alone to support a wider range of Solidity versions and just bump the foundry solc? That said, we are already using features in e.g. 0.8.11 ( |
fd1e7f3
to
5817aab
Compare
closes #1448
closes #833