Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

chore(ci): try to cache reth and go submodules in CI. Remove claude code#3037

Merged
theochap merged 2 commits intomainfrom
theo/cache-reth-go-deps
Nov 14, 2025
Merged

chore(ci): try to cache reth and go submodules in CI. Remove claude code#3037
theochap merged 2 commits intomainfrom
theo/cache-reth-go-deps

Conversation

@theochap
Copy link
Copy Markdown
Member

@theochap theochap commented Nov 14, 2025

Description

Tries to cache cargo build dependencies for reth as a submodule in CI
Tries to cache build dependencies for go submodules in CI
Remove claude code

Copilot AI review requested due to automatic review settings November 14, 2025 13:38
@claude
Copy link
Copy Markdown
Contributor

claude bot commented Nov 14, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds caching configuration for the reth submodule's Cargo build dependencies in CI workflows to improve build times.

Key Changes

  • Adds workspaces parameter to the Swatinem/rust-cache action configuration
  • Configures caching for both the main workspace and the reth submodule workspace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@theochap theochap changed the title chore(ci): try to cache reth submodule in CI chore(ci): try to cache reth and go submodules in CI. Remove claude code Nov 14, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.1%. Comparing base (b192955) to head (91eba56).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@theochap theochap enabled auto-merge November 14, 2025 14:46
@theochap theochap disabled auto-merge November 14, 2025 15:01
@theochap theochap merged commit be9d673 into main Nov 14, 2025
42 of 46 checks passed
@theochap theochap deleted the theo/cache-reth-go-deps branch November 14, 2025 15:01
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
…ode (op-rs/kona#3037)

## Description

Tries to cache cargo build dependencies for reth as a submodule in CI
Tries to cache build dependencies for go submodules in CI
Remove claude code
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
…ode (op-rs/kona#3037)

## Description

Tries to cache cargo build dependencies for reth as a submodule in CI
Tries to cache build dependencies for go submodules in CI
Remove claude code
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants