Skip to content

Update formal-ledger-specifications#5162

Merged
carlostome merged 1 commit intomasterfrom
carlos/update-fls
Jul 14, 2025
Merged

Update formal-ledger-specifications#5162
carlostome merged 1 commit intomasterfrom
carlos/update-fls

Conversation

@carlostome
Copy link
Contributor

@carlostome carlostome commented Jul 11, 2025

Description

Conformance is broken wrt master in fls since we moved TxSize from TxBody to Tx in the formal spec (PR #818).

This PR updates the pointer to fls and fixes the build.

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.

@carlostome carlostome marked this pull request as ready for review July 14, 2025 09:08
@carlostome carlostome requested a review from a team as a code owner July 14, 2025 09:08
Copy link
Contributor

@Soupstraw Soupstraw left a comment

Choose a reason for hiding this comment

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

Looks good

@carlostome carlostome enabled auto-merge (squash) July 14, 2025 13:56
@carlostome carlostome disabled auto-merge July 14, 2025 14:02
@carlostome carlostome enabled auto-merge (squash) July 14, 2025 14:03
@carlostome carlostome merged commit 1a163db into master Jul 14, 2025
132 checks passed
@carlostome carlostome deleted the carlos/update-fls branch July 14, 2025 15:57
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