Governor

[Amxx]

Adding code for a modular governance system

PR Checklist

• Tests
• Documentation
• Changelog entry

[Amxx]

Following you comment I did change the compatibility layer's propose function to use a strict inequality

        require(
            getVotes(msg.sender, block.number - 1) > proposalThreshold(),
            "GovernorCompatibilityBravo: proposer votes below proposal threshold"
        );

I really don't like the new behavior, because even which a proposalThreshold() being 0, proposer still need to have some voting power, and for that they need to delegate votes. I personally believe that non-strict comparison would be better here!

[frangio]

The point about the threshold being 0 is a good one. I'm ok with using >= I don't think this impacts compatibility in any meaningful way.