Skip to content

Gauge Proxy V3 Initial Review#142

Open
auroter wants to merge 25 commits intomasterfrom
gauge-proxy
Open

Gauge Proxy V3 Initial Review#142
auroter wants to merge 25 commits intomasterfrom
gauge-proxy

Conversation

@auroter
Copy link

@auroter auroter commented Jan 8, 2022

Created GaugeProxyV3 which contains bug fixes for voting, deprecating vote, poking, and constructing with governance. Created GaugeProxyV3 test script which tests new GaugeProxyV3 fixes along with soft migration.

This is a draft pull request for initial review. The next step will be to modularize the test and use this code as the basis for a soft migration script which does not rely on impersonation.

@auroter auroter marked this pull request as ready for review January 14, 2022 00:28
saito-sv
saito-sv previously approved these changes Jan 14, 2022
Copy link

@saito-sv saito-sv left a comment

Choose a reason for hiding this comment

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

Not need to address my comments, but would great for consistency.

@saito-sv saito-sv self-requested a review January 14, 2022 01:56
@fuzzylemma
Copy link

Tests dont use chai's expects or structured in describe and its, so npx hardhat test test/gauge-proxy-v3.js does not run any tests.

@auroter auroter requested review from bigwampa and jomarip February 5, 2022 01:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants