Skip to content

UTxO-HD for node 10.4#774

Merged
Jimbo4350 merged 1 commit intomasterfrom
utxo-hd-10.4
Apr 17, 2025
Merged

UTxO-HD for node 10.4#774
Jimbo4350 merged 1 commit intomasterfrom
utxo-hd-10.4

Conversation

@jasagredo
Copy link
Copy Markdown
Contributor

@jasagredo jasagredo commented Mar 6, 2025

Changelog

- description: |
    Implement the changes required for UTxO-HD. See the documentation https://ouroboros-consensus.cardano.intersectmbo.org/docs/for-developers/utxo-hd/Overview.
    - Augment `Cardano.Api.Internal.LedgerState.LedgerState` with Cardano `LedgerTables`.
    - Require `SingI` for `QueryFootprint`s in Consensus queries
# uncomment types applicable to the change:
  type:
  - feature        # introduces a new feature
  - breaking       # the API has changed in a breaking way
  # - compatible     # the API has changed but is non-breaking
  # - optimisation   # measurable performance improvements
  # - refactoring    # QoL changes
  # - bugfix         # fixes a defect
  # - test           # fixes/modifies tests
  # - maintenance    # not directly related to the code
  # - release        # related to a new release preparation
  # - documentation  # change in code docs, haddocks...

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

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. See Running tests for more details
  • Self-reviewed the diff

@jasagredo jasagredo force-pushed the utxo-hd-10.4 branch 2 times, most recently from 904b19a to 6cd7889 Compare March 6, 2025 15:15
Base automatically changed from js/10-3-queries to aniketd/release-10-3 March 7, 2025 10:57
@aniketd aniketd force-pushed the aniketd/release-10-3 branch 2 times, most recently from 632c936 to 8763425 Compare March 27, 2025 10:40
@Jimbo4350 Jimbo4350 force-pushed the aniketd/release-10-3 branch from 8763425 to b82a3e3 Compare March 28, 2025 13:07
Base automatically changed from aniketd/release-10-3 to master March 28, 2025 14:45
@jasagredo jasagredo force-pushed the utxo-hd-10.4 branch 4 times, most recently from 8a8d080 to 64fc1b5 Compare April 10, 2025 07:58
@jasagredo jasagredo changed the title DONOTMERGE rebased UTxO-HD 10.4 UTxO-HD for node 10.4 Apr 10, 2025
@jasagredo jasagredo marked this pull request as ready for review April 10, 2025 08:03
@jasagredo jasagredo force-pushed the utxo-hd-10.4 branch 3 times, most recently from 3903590 to fdfbe75 Compare April 16, 2025 18:04
Copy link
Copy Markdown
Contributor

@Jimbo4350 Jimbo4350 left a comment

Choose a reason for hiding this comment

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

LGTM, one minor comment.

Comment thread cardano-api/src/Cardano/Api/Internal/LedgerState.hs Outdated
@Jimbo4350 Jimbo4350 enabled auto-merge April 17, 2025 14:22
@Jimbo4350 Jimbo4350 added this pull request to the merge queue Apr 17, 2025
Merged via the queue into master with commit 29793b3 Apr 17, 2025
29 checks passed
@Jimbo4350 Jimbo4350 deleted the utxo-hd-10.4 branch April 17, 2025 15:21
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.

3 participants