⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

chore(builder): Lift op-rbuilder Binary #385

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
refcell wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore(builder): Lift op-rbuilder Binary #385

refcell wants to merge 2 commits into from
+102 −56

Conversation

@refcell
Copy link
Contributor

@refcell refcell commented Jan 12, 2026

Summary

Moves the op-rbuilder binary from crates/builder/op-rbuilder/src/bin/ to the top-level bin/op-rbuilder/ directory, aligning with the existing bin/node structure.

Closes #374

Changes

  • Add bin/op-rbuilder/ package with inlined launch logic
  • Remove binary definition from op-rbuilder crate (library remains unchanged)
  • Remove unused launch() function from launcher.rs

@refcell refcell self-assigned this Jan 12, 2026
@cb-heimdall
Copy link
Collaborator

cb-heimdall commented Jan 12, 2026 •
edited
Loading

🟡 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 1
Sum 2

@refcell refcell added K-enhancement Kind: New feature or request A-builder Area: builder crates A-bin Area: binary related labels Jan 12, 2026
@refcell refcell added the F-do-not-merge Flag: don't merge until this label is removed. label Jan 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danyalprout danyalprout Awaiting requested review from danyalprout

@haardikk21 haardikk21 Awaiting requested review from haardikk21

At least 1 approving review is required to merge this pull request.

Assignees

@refcell refcell

Labels

A-bin Area: binary related A-builder Area: builder crates F-do-not-merge Flag: don't merge until this label is removed. K-enhancement Kind: New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

chore(builder): Binary Lift

3 participants

@refcell @cb-heimdall