Core(Queue): Update to use Aragon Protocol

[izqui]

No description provided.

[sohkai]

Couple of notes for now, I think the logic looks good but will review again with the rest of the contracts.

[CLAassistant]

All committers have signed the CLA.

[codecov-io]

Codecov Report

Merging #195 (a0ba123) into master (53f174f) will decrease coverage by 0.49%.
The diff coverage is 2.38%.

@@            Coverage Diff             @@
##           master     #195      +/-   ##
==========================================
- Coverage   51.56%   51.07%   -0.50%     
==========================================
  Files          31       31              
  Lines         510      513       +3     
  Branches       58       59       +1     
==========================================
- Hits          263      262       -1     
- Misses        247      251       +4     

Impacted Files	Coverage Δ
packages/erc3k/contracts/test/ERC3000Mock.sol	0.00% <0.00%> (ø)
...ackages/govern-core/contracts/test/ERC3000Mock.sol	0.00% <0.00%> (ø)
...es/govern-core/contracts/pipelines/GovernQueue.sol	9.27% <2.63%> (-1.60%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 53f174f...a0ba123. Read the comment docs.

[sohkai]

@izqui @nivida I've added a few commits to polish a few rough edges.

The only logical change was with a0ba123, which fixes the submitter's collateral being stuck in the case of a veto on a challenged action. Like rejection, I felt it most appropriate to return this collateral to the challenger.