Skip to content

Fix Dijkstra-era script purpose rendering#6475

Open
Jimbo4350 wants to merge 1 commit intomasterfrom
fix/dijkstra-script-purpose-rendering
Open

Fix Dijkstra-era script purpose rendering#6475
Jimbo4350 wants to merge 1 commit intomasterfrom
fix/dijkstra-script-purpose-rendering

Conversation

@Jimbo4350
Copy link
Copy Markdown
Contributor

Description

Replace undefined with renderConwayPlutusPurpose for AlonzoEraOnwardsDijkstra in renderScriptPurpose, preventing a runtime crash when tracing script purposes in the Dijkstra era.

Based on #6427 by @Savissy.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated.
  • Any changes are noted in the CHANGELOG.md for affected package
  • The version bounds in .cabal files are updated
  • CI passes.
  • Self-reviewed the diff

@Jimbo4350 Jimbo4350 force-pushed the fix/dijkstra-script-purpose-rendering branch 3 times, most recently from ee9b7ef to 61f6fee Compare March 25, 2026 12:49
@Jimbo4350 Jimbo4350 enabled auto-merge March 25, 2026 12:56
@Jimbo4350 Jimbo4350 force-pushed the fix/dijkstra-script-purpose-rendering branch from 61f6fee to 8061c70 Compare March 31, 2026 12:56
@Jimbo4350 Jimbo4350 added this pull request to the merge queue Mar 31, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Mar 31, 2026
@Jimbo4350 Jimbo4350 added this pull request to the merge queue Mar 31, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 31, 2026
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.

2 participants