Skip to content

Conversation

geo2a
Copy link
Contributor

@geo2a geo2a commented Apr 2, 2025

This PR extracts some small changes from #1288 in order to make the rebase of #1288 onto main more manageable.

@geo2a geo2a changed the title Geo2a/split up pr 1288 Split out ImmutableEraParams Apr 2, 2025
@geo2a geo2a self-assigned this Apr 2, 2025
@geo2a geo2a force-pushed the geo2a/split-up-pr-1288 branch from 15aea3f to 95683bb Compare April 2, 2025 14:48
Copy link
Member

@amesgen amesgen left a comment

Choose a reason for hiding this comment

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

LGTM, apart from formatting and the commit message of 73db2b7 still referring to "this PR"

nfrisby added 3 commits April 2, 2025 15:54
Add ImmutableEraParams instances for test blocks without needing
the SingleEraBlock omnibus.
GHC 9.4+ "erroneously" requires HasHeader blk. This patch prevents that. I
_think_ the new local signature forces GHC to invoke the HeaderHash (Header
blk) ~ HeaderHash blk rewrite _later_, thereby avoiding that undesirable Wanted
constraint.
It's declared upstream in Control.ResourceRegistry now.
@geo2a geo2a force-pushed the geo2a/split-up-pr-1288 branch from 95683bb to ff188ac Compare April 2, 2025 14:56
@geo2a geo2a enabled auto-merge April 2, 2025 14:57
@geo2a geo2a force-pushed the geo2a/split-up-pr-1288 branch from ff188ac to 0191ab4 Compare April 2, 2025 15:52
@geo2a geo2a added this pull request to the merge queue Apr 2, 2025
Merged via the queue into main with commit b0a9ee2 Apr 2, 2025
20 checks passed
@geo2a geo2a deleted the geo2a/split-up-pr-1288 branch April 2, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants