Skip to content

fix: use base/base main branch for base-reth-node builds#181

Merged
meyer9 merged 1 commit into
mainfrom
fix/ci-use-main-for-base-reth-node
May 14, 2026
Merged

fix: use base/base main branch for base-reth-node builds#181
meyer9 merged 1 commit into
mainfrom
fix/ci-use-main-for-base-reth-node

Conversation

@meyer9
Copy link
Copy Markdown
Collaborator

@meyer9 meyer9 commented May 14, 2026

Summary

  • Reverts base_reth_node_version from short commit SHA (1e4a8a7) back to main in both versions.env and _build-binaries.yaml
  • Simplifies checkout logic in build-base-reth-node.sh to match build-reth.sh

Root Cause

PR #175 changed BASE_RETH_NODE_VERSION from main to the short SHA 1e4a8a7. GitHub does not allow fetching arbitrary commit SHAs as refs, so git fetch origin 1e4a8a7 fails during CI with:

fatal: couldn't find remote ref 1e4a8a7

The commit exists in base/base, but Git's protocol doesn't support fetching by raw SHA from GitHub remotes.

Changes

  1. clients/versions.envBASE_RETH_NODE_VERSION back to main
  2. .github/workflows/_build-binaries.yaml — default input back to main
  3. clients/build-base-reth-node.sh — removed git fetch origin "$VERSION" || true + fallback checkout; now uses a single git checkout -f "$VERSION" matching the pattern in build-reth.sh

Revert base_reth_node_version from short commit SHA (1e4a8a7) back to
'main'. GitHub does not allow fetching arbitrary commit SHAs as refs,
so `git fetch origin 1e4a8a7` fails during CI.

Also simplify the checkout logic in build-base-reth-node.sh to match
build-reth.sh — a direct `git checkout -f` is sufficient for both
branch names and commit SHAs present in the cloned history.
@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented May 14, 2026

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/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

@meyer9 meyer9 enabled auto-merge (squash) May 14, 2026 15:33
@meyer9 meyer9 merged commit cb5b9d6 into main May 14, 2026
14 checks passed
@meyer9 meyer9 deleted the fix/ci-use-main-for-base-reth-node branch May 14, 2026 15:55
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