Fail PlutusV1-V3 translation for non-empty subTransactionsTxBody#5669
Merged
Fail PlutusV1-V3 translation for non-empty subTransactionsTxBody#5669
subTransactionsTxBody#5669Conversation
Lucsanszky
changed the base branch from
master
to
lehins/memoize-transaction-validation
Lucsanszky
changed the title
subTransactionsTxBody
Lucsanszky
commented
Mar 23, 2026
lehins
force-pushed
the
lehins/memoize-transaction-validation
branch
6 times, most recently
from
adef9ae to
7e175ac
Compare
lehins
force-pushed
the
lehins/memoize-transaction-validation
branch
3 times, most recently
from
3b5a330 to
3594844
Compare
Base automatically changed from
lehins/memoize-transaction-validation
to
master
April 24, 2026 11:24
Lucsanszky
force-pushed
the
ldan/subtransactions-fail-plutus-translation
branch
from
e1da532 to
e1c4383
Compare
10 tasks
lehins
requested changes
May 5, 2026
Collaborator
lehins
left a comment
lehins
left a comment
There was a problem hiding this comment.
Yes, we need a new failure
Lucsanszky
force-pushed
the
ldan/subtransactions-fail-plutus-translation
branch
from
e1c4383 to
67d3906
Compare
Lucsanszky
commented
May 5, 2026
teodanciu
reviewed
May 5, 2026
Contributor
teodanciu
left a comment
teodanciu
left a comment
There was a problem hiding this comment.
Looks good to me, will let Alexey approve it since he had requested changes.
lehins
requested changes
May 6, 2026
Collaborator
lehins
left a comment
lehins
left a comment
There was a problem hiding this comment.
Aside from usage of partial function this PR look good.
Lucsanszky
force-pushed
the
ldan/subtransactions-fail-plutus-translation
branch
from
131b1f4 to
8b60622
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Resolves #5633
Checklist
CHANGELOG.mdfiles updated for packages with externally visible changes.NOTE: New section is never added with the code changes. (See RELEASING.md).
.cabalandCHANGELOG.mdfiles when necessary, according to theversioning process.
.cabalfiles updated when necessary.NOTE: If bounds change in a cabal file, that package itself must have a version increase. (See RELEASING.md).
scripts/fourmolize.sh).scripts/cabal-format.sh).scripts/gen-cddl.sh)hie.yamlupdated (usescripts/gen-hie.sh).