Skip to content

Integrate newer cardano-ledger version #1525

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Integrate newer cardano-ledger version #1525

wants to merge 18 commits into from

Conversation

jasagredo
Copy link
Contributor

@jasagredo jasagredo commented May 27, 2025 •
edited
Loading

  • Integrates a newer release of cardano-ledger, before the Dijkstra era
  • Uses a custom ouroboros-network that just adds NodeToClientV_21, which eventually will be replaced with a proper release of network.

amesgen
amesgen reviewed May 27, 2025
View reviewed changes
@jasagredo jasagredo force-pushed the js/newer-ledger branch 3 times, most recently from a466e66 to a776503 Compare May 28, 2025 09:17
@ana-pantilie ana-pantilie mentioned this pull request May 28, 2025
Draft
@jasagredo jasagredo mentioned this pull request May 29, 2025
Closed
@jasagredo jasagredo self-assigned this Jun 2, 2025
@jasagredo jasagredo moved this to 🚫 Help needed in Consensus Team Backlog Jun 2, 2025
@jasagredo jasagredo requested review from lehins and removed request for jorisdral June 2, 2025 15:02
lehins
lehins reviewed Jun 3, 2025
View reviewed changes
Copy link
Contributor

@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 good 👍

@jasagredo jasagredo moved this from 🚫 Help needed to 👀 In review in Consensus Team Backlog Jun 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lehins lehins lehins left review comments

@amesgen amesgen amesgen left review comments

@nfrisby nfrisby Awaiting requested review from nfrisby nfrisby is a code owner

@fraser-iohk fraser-iohk Awaiting requested review from fraser-iohk fraser-iohk is a code owner

@dnadales dnadales Awaiting requested review from dnadales dnadales is a code owner

@geo2a geo2a Awaiting requested review from geo2a geo2a is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@jasagredo jasagredo

Labels
None yet
Projects
Consensus Team Backlog
Status: 👀 In review
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jasagredo @lehins @amesgen @geo2a