removes the confusing (and unlawful) mintsListIso. We can just use txSkelMintsToList and txSkelMintsFromList, without the mental overhead associated with a non-Iso.
changes the type of txSkelMintsFromList and similar functions by replacing NonZero Integer with Integer. Since we have to check whether things add up to zero anyway once we start using addToTxSkelMints (which txSkelMintsFromList and the Semigroup instance of TxSkelMints do), the NonZero type doesn't really enforce a valuable invariant, and only adds noise to the interface.
This PR
mintsListIso
. We can just usetxSkelMintsToList
andtxSkelMintsFromList
, without the mental overhead associated with a non-Iso
.txSkelMintsFromList
and similar functions by replacingNonZero Integer
withInteger
. Since we have to check whether things add up to zero anyway once we start usingaddToTxSkelMints
(whichtxSkelMintsFromList
and theSemigroup
instance ofTxSkelMints
do), theNonZero
type doesn't really enforce a valuable invariant, and only adds noise to the interface.