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.

Sign up for GitHub

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

Jump to bottom

upgrade GovernorBravoDelegateG2 to 0.8.10 #208

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

upgrade GovernorBravoDelegateG2 to 0.8.10 #208

wants to merge 1 commit into from
+37 −24

Conversation

jbeanz
Copy link

@jbeanz jbeanz commented Jun 23, 2022

GOAL: upgrade GovernorBravoDelegateG2.sol to 0.8.10

The changes in the function propose were copied from the previous GovernorBravoDelegateG1.sol

@ZrowGz
Copy link

ZrowGz commented Jun 23, 2022

Just wanted to point out that this was completed in the PR I have open (#207):
#207

I also updated error handling throughout the whole protocol. Would love to get another set of eyes on that work too.

@arjun-io arjun-io mentioned this pull request Jul 8, 2022
Closed
eztub
eztub approved these changes Aug 15, 2023
View reviewed changes
Copy link

@eztub eztub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nicholas4455 Nicholas4455 Nicholas4455 approved these changes

@eztub eztub eztub approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@jbeanz @ZrowGz @Nicholas4455 @eztub