It turns out that in fact we did not need to pass around both Ledger.TxOut and Api.TxOut. For clarity, this PR removes the former, leaving the codebase with two distinct TxOut:
Plutus TxOut that is used throughout the codebase, in our mockchain and skeletons
Cardano TxOut that is used during transaction generation
This does not change the behavior of cooked in any way. It will just lighten and clarify the codebase.
This PR closes #441
It turns out that in fact we did not need to pass around both
Ledger.TxOutandApi.TxOut. For clarity, this PR removes the former, leaving the codebase with two distinctTxOut:TxOutthat is used throughout the codebase, in our mockchain and skeletonsTxOutthat is used during transaction generationThis does not change the behavior of cooked in any way. It will just lighten and clarify the codebase.
Note that this PR is forked from PR #431