Skip to content

Commit

Permalink
Merge pull request #817 from IntersectMBO/smelc/remove-useless-output…
Browse files Browse the repository at this point in the history
…-options-parameter

Remove the unused output options from runTxBuild
  • Loading branch information
smelc authored Jul 3, 2024
2 parents 4064194 + 61576b9 commit e29e335
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions cardano-cli/src/Cardano/CLI/EraBased/Run/Transaction.hs
Original file line number Diff line number Diff line change
Expand Up @@ -215,7 +215,6 @@ runTransactionBuildCmd
requiredSigners txAuxScripts txMetadata mProp mOverrideWitnesses votingProceduresAndMaybeScriptWits
proposals
(unFeatured <$> featuredCurrentTreasuryValueM) treasuryDonation
buildOutputOptions

-- TODO: Calculating the script cost should live as a different command.
-- Why? Because then we can simply read a txbody and figure out
Expand Down Expand Up @@ -753,16 +752,15 @@ runTxBuild :: ()
-> [(Proposal era, Maybe (ScriptWitness WitCtxStake era))]
-> Maybe TxCurrentTreasuryValue
-> Maybe TxTreasuryDonation
-> TxBuildOutputOptions
-> ExceptT TxCmdError IO (BalancedTxBody era)
runTxBuild
sbe socketPath networkId mScriptValidity
inputsAndMaybeScriptWits readOnlyRefIns txinsc mReturnCollateral mTotCollateral txouts
(TxOutChangeAddress changeAddr) valuesWithScriptWits mLowerBound mUpperBound
certsAndMaybeScriptWits withdrawals reqSigners txAuxScripts txMetadata
txUpdateProposal mOverrideWits votingProcedures proposals
mCurrentTreasuryValue mTreasuryDonation
_outputOptions = shelleyBasedEraConstraints sbe $ do
mCurrentTreasuryValue mTreasuryDonation =
shelleyBasedEraConstraints sbe $ do

-- TODO: All functions should be parameterized by ShelleyBasedEra
-- as it's not possible to call this function with ByronEra
Expand Down

0 comments on commit e29e335

Please sign in to comment.