delvtech / council

Flexible DAO governance smart contracts, developed by Delv.
Apache License 2.0
93 stars 19 forks source link

Inconsistent use of largeSpend vs highSpend in Spender.sol #53

Closed jrhea closed 2 years ago

jrhea commented 2 years ago

The method is called largeSpend, but the contract variables are highSpendLimit:

https://github.com/element-fi/council/blob/d371ac82184fc26e19e958bd0b861f83e74b2955/contracts/features/Spender.sol#L74

The comment calls it large spend limit even though the param is highSpendLimit:

https://github.com/element-fi/council/blob/d371ac82184fc26e19e958bd0b861f83e74b2955/contracts/features/Spender.sol#L32