Skip to content

backport psbFromByteStringM and psbToByteArray to PinnedSizedBytes to 2.3.3.0#669

Merged
lehins merged 2 commits into
release/cardano-crypto-class-2.3.2.0from
sl/fix-cardano-crypto-wallet-backport
Jul 8, 2026
Merged

backport psbFromByteStringM and psbToByteArray to PinnedSizedBytes to 2.3.3.0#669
lehins merged 2 commits into
release/cardano-crypto-class-2.3.2.0from
sl/fix-cardano-crypto-wallet-backport

Conversation

@disassembler

Copy link
Copy Markdown
Contributor

These were introduced in a newer commit that the cardano-crypto-wallet branch depends on but is not yet present in the tagged base this branch is rebased onto.

Description

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated
  • All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages.
    New section is never added with the code changes. (See RELEASING.md)
  • When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
    versioning process.
  • The version bounds in .cabal files for all affected packages are updated.
    If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
  • Self-reviewed the diff

@disassembler disassembler requested a review from lehins as a code owner June 4, 2026 18:46
These were introduced in a newer commit that the wallet-cbits branch
depends on but is not yet present in the tagged base this branch is
rebased onto.
@disassembler disassembler force-pushed the sl/fix-cardano-crypto-wallet-backport branch from 73f7f1f to bcd0bcc Compare June 4, 2026 18:54
@disassembler disassembler changed the base branch from master to release/cardano-crypto-class-2.3.2.0 June 4, 2026 18:55
@disassembler disassembler changed the title Add psbFromByteStringM and psbToByteArray to PinnedSizedBytes backport psbFromByteStringM and psbToByteArray to PinnedSizedBytes to 2.3.2.0 Jun 4, 2026

@lehins lehins left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It is a bit unfortunate that there were already two major versions 2.4.0.0 and 2.5.0.0 released since 2.3.2.0, but conceptually it is not wrong to release 2.3.3.0 version and pretend like this functionality was removed in 2.4.0.0 only to be brought back in 2.5.1.0

Please make sure to bump up the version in cabal file and add a new section in the changelog with 2.3.3.0 header with changelog entries from 73339f4 and 31402fc

Other than that it looks fine to make a backport like this

@lehins lehins changed the title backport psbFromByteStringM and psbToByteArray to PinnedSizedBytes to 2.3.2.0 backport psbFromByteStringM and psbToByteArray to PinnedSizedBytes to 2.3.3.0 Jul 6, 2026
Add changelog entries and version bump for psbToByteArray and
psbFromByteStringM, backported from 2.5.1.0.

@lehins lehins left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍

@lehins lehins merged commit 30b0259 into release/cardano-crypto-class-2.3.2.0 Jul 8, 2026
21 of 24 checks passed
@lehins lehins deleted the sl/fix-cardano-crypto-wallet-backport branch July 8, 2026 15:58
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