Skip to content

Conversation

jasagredo
Copy link
Contributor

Description

Add your description here, if it fixes a particular issue please provide a
link
to the issue.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. These may include:
    • golden tests
    • property tests
    • roundtrip tests
    • integration tests
      See Runnings tests for more details
  • Any changes are noted in the CHANGELOG.md for affected package
  • The version bounds in .cabal files are updated
  • CI passes. See note on CI. The following CI checks are required:
    • Code is linted with hlint. See .github/workflows/check-hlint.yml to get the hlint version
    • Code is formatted with stylish-haskell. See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
    • Code builds on Linux, MacOS and Windows for ghc-9.6 and ghc-9.12
  • Self-reviewed the diff

Note on CI

If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG node developers to do this
for you.

@jasagredo jasagredo force-pushed the js/more-consensus-10.6 branch from 717ad77 to 650e6b8 Compare October 2, 2025 16:11
@crocodile-dentist crocodile-dentist force-pushed the ana/10.6-final-integration-mix branch 4 times, most recently from 847992c to c6f48e8 Compare October 2, 2025 18:19
@jasagredo jasagredo force-pushed the js/more-consensus-10.6 branch 5 times, most recently from 3104cc8 to a2da3a9 Compare October 3, 2025 10:33
Copy link
Contributor

@fraser-iohk fraser-iohk left a comment

Choose a reason for hiding this comment

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

can't speak to the Dijkstra-related changed but the tracing changes are all good 👍

@ana-pantilie ana-pantilie force-pushed the js/more-consensus-10.6 branch from a2da3a9 to 6594708 Compare October 7, 2025 17:58
@ana-pantilie ana-pantilie marked this pull request as ready for review October 7, 2025 17:59
@ana-pantilie ana-pantilie requested review from a team as code owners October 7, 2025 17:59
@ana-pantilie ana-pantilie merged commit eeb0a0d into ana/10.6-final-integration-mix Oct 7, 2025
8 of 14 checks passed
@ana-pantilie ana-pantilie deleted the js/more-consensus-10.6 branch October 7, 2025 17:59
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