Skip to content

Conversation

Soupstraw
Copy link
Contributor

@Soupstraw Soupstraw commented Aug 13, 2025

Description

Adds DijkstraPlutusPurpose with the new guarding purpose. I also removed the possibility to represent redeemers as a flat array in CDDL, since the comment said it should be removed in Dijkstra.

close #5204
close #5205

Checklist

  • Commits in meaningful sequence and with useful messages.
  • Tests added or updated when needed.
  • CHANGELOG.md files updated for packages with externally visible changes.
    NOTE: New section is never added with the code changes. (See RELEASING.md).
  • Versions updated in .cabal and CHANGELOG.md files when necessary, according to the
    versioning process.
  • Version bounds in .cabal files updated when necessary.
    NOTE: If bounds change in a cabal file, that package itself must have a version increase. (See RELEASING.md).
  • Code formatted (use scripts/fourmolize.sh).
  • Cabal files formatted (use scripts/cabal-format.sh).
  • CDDL files are up to date (use scripts/gen-cddl.sh)
  • hie.yaml updated (use scripts/gen-hie.sh).
  • Self-reviewed the diff.

@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch 4 times, most recently from ef40340 to 911a522 Compare August 15, 2025 11:38
@Soupstraw Soupstraw changed the title Added DijkstraPlutusScript Added DijkstraPlutusPurpose Aug 15, 2025
@Soupstraw Soupstraw marked this pull request as ready for review August 27, 2025 12:33
@Soupstraw Soupstraw requested a review from a team as a code owner August 27, 2025 12:33
@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch 2 times, most recently from 41377e4 to 82f0920 Compare August 27, 2025 13:27
@Soupstraw Soupstraw marked this pull request as draft August 27, 2025 13:28
@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch 9 times, most recently from 0bbf3ad to 810a5f5 Compare August 28, 2025 11:04
@Soupstraw Soupstraw marked this pull request as ready for review August 28, 2025 11:05
@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch from 810a5f5 to 7f48957 Compare August 28, 2025 11:06
@Soupstraw Soupstraw requested a review from lehins August 28, 2025 11:29
@Soupstraw Soupstraw marked this pull request as draft August 28, 2025 12:20
@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch from 7f48957 to 8fa6fd3 Compare August 28, 2025 12:28
@Soupstraw Soupstraw marked this pull request as ready for review August 28, 2025 12:34
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thank you!

@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch from bd67dcb to 6e87aa7 Compare September 1, 2025 09:17
@Soupstraw Soupstraw force-pushed the jj/dijkstra-plutuspurpose branch from 6e87aa7 to ed7a81a Compare September 1, 2025 09:21
@Soupstraw Soupstraw enabled auto-merge (rebase) September 1, 2025 09:21
@Soupstraw Soupstraw merged commit 74e2b99 into master Sep 1, 2025
121 of 122 checks passed
@Soupstraw Soupstraw deleted the jj/dijkstra-plutuspurpose branch September 1, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add new Guarding plutus purpose to PlutusV4 script context Add Dijkstra era PlutusPurpose
2 participants