Skip to content

Heze fork boilerplate#9573

Open
eserilev wants to merge 19 commits into
sigp:unstablefrom
eserilev:heze-fork-boilerplate
Open

Heze fork boilerplate#9573
eserilev wants to merge 19 commits into
sigp:unstablefrom
eserilev:heze-fork-boilerplate

Conversation

@eserilev

@eserilev eserilev commented Jul 4, 2026

Copy link
Copy Markdown
Member

Add fork boilerplate for Heze

Written by AI, but I did a self review

@eserilev
eserilev requested a review from jxs as a code owner July 4, 2026 07:40
@eserilev eserilev added the heze label Jul 4, 2026
@eserilev
eserilev force-pushed the heze-fork-boilerplate branch 2 times, most recently from 86db003 to 9a3ea67 Compare July 4, 2026 19:16
@eserilev eserilev added the ready-for-review The code is ready for review label Jul 5, 2026
@mergify

mergify Bot commented Jul 5, 2026

Copy link
Copy Markdown

Some required checks have failed. Could you please take a look @eserilev? 🙏

@mergify mergify Bot added waiting-on-author The reviewer has suggested changes and awaits thier implementation. and removed ready-for-review The code is ready for review labels Jul 5, 2026
@mergify mergify Bot added ready-for-review The code is ready for review and removed waiting-on-author The reviewer has suggested changes and awaits thier implementation. labels Jul 7, 2026

@rahulbarmann rahulbarmann left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewed the boilerplate, looks solid. Just one small nit inline.

Comment thread consensus/types/src/light_client/light_client_header.rs Outdated
let extra_fields = get_extra_fields(spec);

if spec.is_gloas_scheduled() {
if spec.is_heze_scheduled() {

@conache conache Jul 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Very small nit and it shouldn't be breaking for this PR, but flagging to keep it under the radar.

We should probably add a separate check for spec.is_heze_scheduled() in the following test helpers:

The tests using these helpers would fail when is_heze_scheduled() returns true, because the expected value becomes ConfigAndPreset::Heze, while the actual value is ConfigAndPreset::Gloas and the assertion would fail.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

heze ready-for-review The code is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants