Skip to content

Conversation

@danyalprout
Copy link
Collaborator

Note

Stacked on top of #679 - not yet ready for review.

@danyalprout danyalprout changed the title chore(builder): combined builder setup chore(builder): consolidated builder setup on shared node Feb 8, 2026
@danyalprout danyalprout force-pushed the danyal/simplify-builder-setup-2 branch from 1098922 to 715de19 Compare February 9, 2026 20:07
@danyalprout danyalprout force-pushed the danyal/simplify-builder-setup-3 branch from 7fc1d9d to 1aa0257 Compare February 9, 2026 20:07
@danyalprout danyalprout force-pushed the danyal/simplify-builder-setup-2 branch from 715de19 to f59e2cf Compare February 9, 2026 21:55
Base automatically changed from danyal/simplify-builder-setup-2 to main February 9, 2026 22:57
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Feb 9, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@danyalprout danyalprout force-pushed the danyal/simplify-builder-setup-3 branch from 1aa0257 to 6d999cb Compare February 11, 2026 18:03
…, move extension

- Fix bug: replace manual EngineNodeLauncher creation with .launch() in
  runner, test_utils, and devnet (delegates to reth's default launcher)
- Undo rename: NodeHooks back to BaseBuilder
- Undo rename: NodeRunner back to BaseNodeRunner
- Move MeteringStoreExtension from bin/builder into base-builder-metering crate

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

2 participants