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

Comments

chore(ga): only run claude once per pull request#2695

Merged
emhane merged 1 commit intomainfrom
theo/less-noisy-claude
Aug 13, 2025
Merged

chore(ga): only run claude once per pull request#2695
emhane merged 1 commit intomainfrom
theo/less-noisy-claude

Conversation

@theochap
Copy link
Member

Description

Claude was getting noisy by commenting for every commit. We just want to have it once per PR

Copilot AI review requested due to automatic review settings August 12, 2025 20:54
Copy link
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 modifies the GitHub Actions workflow to reduce noise by limiting Claude code review to run only when a pull request is opened, rather than on every commit push to the PR.

  • Removes the synchronize trigger from the pull request workflow
  • Maintains the opened trigger to ensure code review still occurs for new PRs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Aug 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.2%. Comparing base (2515144) to head (884907b).
⚠️ Report is 7 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.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@theochap theochap force-pushed the theo/less-noisy-claude branch from 72aa950 to 884907b Compare August 12, 2025 20:58
@theochap theochap self-assigned this Aug 12, 2025
@theochap theochap added the A-node Area: cl node (eq. Go op-node) handles single-chain consensus label Aug 12, 2025
@theochap theochap added this pull request to the merge queue Aug 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 13, 2025
@emhane emhane merged commit 354d316 into main Aug 13, 2025
33 checks passed
@emhane emhane deleted the theo/less-noisy-claude branch August 13, 2025 12:30
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
## Description

Claude was getting noisy by commenting for every commit. We just want to
have it once per PR
theochap added a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
## Description

Claude was getting noisy by commenting for every commit. We just want to
have it once per PR
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A-node Area: cl node (eq. Go op-node) handles single-chain consensus

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants