-
-
Notifications
You must be signed in to change notification settings - Fork 254
Release 129.0.0 #4082
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
Release 129.0.0 #4082
Conversation
a3fb3a4 to
36de4b0
Compare
|
Could this release maybe wait to include some or all of the following PRs you currently have open? |
Hey @MajorLift !
|
|
If this is urgent, we can definitely go ahead without waiting. Batching updates together is preferable, but keeping this PR open for too long and blocking other releases is also a concern. I'll leave it up to you to merge this or wait to include #4030 based on your timeline. |
3c01548 to
2c2c1f6
Compare
mcmire
left a comment
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.
I'm okay with releasing the changes outlined here for now and releasing more later, but I did have a comment about one of the changes.
fabcd63 to
870518d
Compare
mcmire
left a comment
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.
This is the release candidate for 129.0.0. It adds an export for const SUPPORTED_CHAIN_IDS.
#4079