treasury: Delta Governor v1 - Githubissues

livepeer / protocol

Livepeer protocol

MIT License

154 stars 45 forks source link

treasury: Delta Governor v1 #615

Closed victorges closed 1 year ago

victorges commented 1 year ago

What does this pull request do? Explain your changes. (required) This is the implementation for the LivepeerGovernor specified in LIP-89. It depends on the BondingVotes contract implementation from #614.

The higher level output of this PR is a LivepeerGovernor contract compatible with the OZ Governor framework, whose participants bonding power is derived from their staked LPT in the same way as LIP-16).

Specific updates (required)

Create governance contracts described in LIP-89 apart from BondingVotes.
Unit tests for GovernorCountingOverridable
Integration tests to LivepeerGovernor incl deployment scripts

How did you test each of these updates (required)

yarn test all the old and new tests
Devnet on Goerli

Does this pull request close any open issues? Implements PRO-31

Checklist:

[ ] README and other documentation updated
[x] All tests using yarn test pass

victorges commented 1 year ago

@yondonfu Also ready for another round! 😁

Notice that I made a "merge" with the parent branch just to avoid rebasing and losing all history. Let me know if you'd prefer a clean rebase. When merging, my plan is to either do the full linearization of the branches or squash&merge them instead not to keep the non-linear git history.