Skip to content

chore(ci): Remove redundant automated review workflow#61

Merged
MarjovanLier merged 1 commit into
mainfrom
ci/remove-claude-review
Jun 27, 2026
Merged

chore(ci): Remove redundant automated review workflow#61
MarjovanLier merged 1 commit into
mainfrom
ci/remove-claude-review

Conversation

@MarjovanLier

Copy link
Copy Markdown
Owner

Summary

Removes the claude-code-review.yml workflow, which ran the Claude Code Review action on every pull request. Its output duplicated the other automated reviewers (CodeRabbit, Qodo, Gemini, Copilot) and added noise to the discussion.

The claude.yml workflow (the @claude mention responder) is kept for on-demand use.

No effect on the build/test pipeline.

The removed action posted an automated review on every pull request,
duplicating the other review bots and adding noise to the discussion.
The mention-triggered responder workflow is left in place for
on-demand use.

Signed-off-by: Marjo Wenzel van Lier <marjo.vanlier@gmail.com>
Copilot AI review requested due to automatic review settings June 27, 2026 04:19
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Jun 27, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)
  • .github/workflows/claude-code-review.yml is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 9308ca93-f7b6-4500-b258-a3235d85791f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch ci/remove-claude-review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@qodo-code-review

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Remove redundant Claude automated PR review workflow
⚙️ Configuration changes 🕐 Less than 5 minutes

Grey Divider

Description

• Remove the always-on Claude code review GitHub Actions workflow for pull requests.
• Reduce PR noise by eliminating a bot that duplicated other automated reviewers.
• Keep the mention-triggered Claude workflow for on-demand reviews.
Diagram

graph TD
  PR["Pull request event"] --> WF["Claude review workflow (removed)"] --> ACT["claude-code-action"] --> CMT["PR comment"]
  PR --> OB["Other review bots"]
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Gate the workflow behind labels or author association
  • ➕ Retains automated Claude reviews for targeted cases (e.g., first-time contributors).
  • ➕ Avoids blanket PR noise while preserving some automation.
  • ➖ More workflow complexity and ongoing tuning.
  • ➖ Still risks unwanted comments if gating logic is too broad.
2. Convert to manual trigger (workflow_dispatch) instead of deletion
  • ➕ Keeps the workflow definition available for maintainers without relying on @mention.
  • ➕ Easier to re-enable without restoring a deleted file.
  • ➖ Adds another way to invoke Claude alongside the existing mention-based workflow.
  • ➖ Still requires maintenance for a workflow that may be rarely used.

Recommendation: The current approach (deleting the always-on review workflow) is the cleanest way to eliminate duplicated automated feedback and reduce PR noise, while still supporting on-demand usage via the existing mention-triggered Claude workflow. The main alternatives (label-gating or manual dispatch) add complexity for limited benefit given the stated goal.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Removes the always-on Claude automated PR review workflow to reduce redundant review noise, while keeping the on-demand @claude responder workflow intact.

Changes:

  • Deleted .github/workflows/claude-code-review.yml, which ran on pull_request events.
  • Retains the existing on-demand Claude workflow (.github/workflows/claude.yml) for comment/mention-triggered usage.

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

@codecov

codecov Bot commented Jun 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (f7485d3) to head (14311c4).

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #61   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
  Complexity        32        32           
===========================================
  Files              1         1           
  Lines             61        61           
===========================================
  Hits              61        61           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@qodo-code-review

qodo-code-review Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

Auto-approved notes - no code review findings
- 'auto_approve_for_no_suggestions' flag enabled

Previous review results

Review updated until commit 14311c4

Results up to commit 14311c4


🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review
Auto-approved notes - no code review findings
- 'auto_approve_for_no_suggestions' flag enabled

Qodo Logo

@qodo-code-review

Copy link
Copy Markdown
Contributor

Changelog updates: 🔄

2026-06-27

Changed

  • Simplified CI by removing a redundant automated pull-request review workflow.

to commit the new content to the CHANGELOG.md file, please type:
'/update_changelog --pr_update_changelog.push_changelog_changes=true'

@qodo-code-review

Copy link
Copy Markdown
Contributor

Code review by qodo was updated up to the latest commit 14311c4

@MarjovanLier MarjovanLier merged commit 89ef147 into main Jun 27, 2026
15 checks passed
@MarjovanLier MarjovanLier deleted the ci/remove-claude-review branch June 27, 2026 04:36
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