-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Plutus V3 script updates #822
Merged
Merged
Conversation
This file contains 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
@Jimbo4350 Thoughts about the renaming proposed in #809 ? |
Looking good 👍 I checked that it is now possible to spend a UTxO locked with PlutusV3 script without providing datum (when the script allows it). |
palas
force-pushed
the
jordan/plutus-v3-fixes
branch
2 times, most recently
from
July 15, 2024 23:08
d3d6570
to
4d77019
Compare
FYI: I have rebased your branch because we have done changes to the formatting. I have made a copy of the unrebased branch that you can find in my previous comment in this PR. |
Jimbo4350
force-pushed
the
jordan/plutus-v3-fixes
branch
3 times, most recently
from
July 24, 2024 11:56
2780429
to
ecb78e6
Compare
Jimbo4350
changed the title
Relax requirement of supplying datums for plutus scripts
Plutus V3 script updates
Jul 24, 2024
Jimbo4350
force-pushed
the
jordan/plutus-v3-fixes
branch
from
July 24, 2024 11:57
ecb78e6
to
b35d043
Compare
Jimbo4350
requested review from
newhoggy,
carbolymer,
smelc,
palas,
kevinhammond and
disassembler
as code owners
July 24, 2024 11:57
for a spending script optional
relax the datum requirement of spending scripts in the conway era
Jimbo4350
force-pushed
the
jordan/plutus-v3-fixes
branch
from
July 24, 2024 11:59
b35d043
to
e053fc3
Compare
… script witnesses
a spending script datum in Conway
- Cardano.CLI.Read - Cardano.CLI.Legacy.Options - Cardano.CLI.EraBased.Options.Transaction
Jimbo4350
force-pushed
the
jordan/plutus-v3-fixes
branch
from
July 24, 2024 12:00
e053fc3
to
7a4271a
Compare
carbolymer
approved these changes
Jul 24, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Changelog
Context
Additional context for the PR goes here. If the PR fixes a particular issue please provide a link to the issue.
How to trust this PR
Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.
Checklist