Skip to content

Update CHaP - #1436

Merged
palas merged 1 commit into
masterfrom
update-chap-in-flake
Sep 2, 2026
Merged

Update CHaP#1436
palas merged 1 commit into
masterfrom
update-chap-in-flake

Conversation

@palas

@palas palas commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Context

This issue is caused by this other issue which has been solved by ouroboros-consensus-4.2.1.0. So this PR updates CHaP to include this fixed version.

How to trust this PR

You can test this issue on this PR. Fixes #1434.

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
  • Changelog fragment added in .changes/

@palas palas self-assigned this Sep 2, 2026
Copilot AI lite review requested due to automatic review settings September 2, 2026 15:45
@palas palas added the bug Something isn't working correctly or outputting unintended results. label Sep 2, 2026
@palas
palas force-pushed the update-chap-in-flake branch from 39cebe6 to 7c25113 Compare September 2, 2026 15:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The new changelog fragment does not follow the repo’s established .changes YAML convention for description formatting and may be interpreted differently by tooling.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Updates the Cardano Haskell Packages (CHaP) snapshot used by cardano-cli so the dependency set includes ouroboros-consensus-4.2.1.0, which addresses the hang described in issue #1434.

Changes:

  • Bump the pinned CHaP Git revision in flake.lock.
  • Update cabal.project index-state for cardano-haskell-packages to match the new snapshot timing.
  • Add a changelog fragment describing the dependency bump and impacted commands.
File summaries
File Description
flake.lock Updates the pinned CHaP revision/hash used by Nix flakes.
cabal.project Advances the CHaP index-state timestamp to align with the updated snapshot.
.changes/20260902_154535_cardano-cli_pablo.lamela_update_chap_in_flake.yml Adds a changelog fragment for the CHaP/dependency bump.
Review details
  • Files reviewed: 2/3 changed files
  • Comments generated: 1
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .changes/20260902_154535_cardano-cli_pablo.lamela_update_chap_in_flake.yml Outdated
@palas
palas force-pushed the update-chap-in-flake branch from 7c25113 to b02965c Compare September 2, 2026 15:51
@palas
palas added this pull request to the merge queue Sep 2, 2026
Merged via the queue into master with commit 42a9a75 Sep 2, 2026
25 checks passed
@palas
palas deleted the update-chap-in-flake branch September 2, 2026 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working correctly or outputting unintended results.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

BUG - query kes-period-info is broken on cli>v11.0.0.0 for node v11.0.x

3 participants