Skip to content

remove claude from ci#221

Merged
tac0turtle merged 1 commit intomainfrom
marko/remove_claude
Apr 21, 2026
Merged

remove claude from ci#221
tac0turtle merged 1 commit intomainfrom
marko/remove_claude

Conversation

@tac0turtle
Copy link
Copy Markdown
Contributor

@tac0turtle tac0turtle commented Apr 21, 2026

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Refactoring

Related Issues

Fixes #(issue)

Checklist

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published

Testing

Additional Notes

Summary by CodeRabbit

  • Chores
    • Removed automated code review workflows from the continuous integration pipeline.

@tac0turtle tac0turtle requested a review from a team as a code owner April 21, 2026 11:33
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 21, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 08d4c9f6-b5fb-4a00-a839-402acf184d0e

📥 Commits

Reviewing files that changed from the base of the PR and between 2cd9dc2 and ac308c5.

📒 Files selected for processing (2)
  • .github/workflows/claude-code-review.yml
  • .github/workflows/claude.yml
💤 Files with no reviewable changes (2)
  • .github/workflows/claude-code-review.yml
  • .github/workflows/claude.yml

📝 Walkthrough

Walkthrough

Two GitHub Actions workflow files were removed: claude-code-review.yml and claude.yml. The former automated code reviews on pull requests; the latter triggered Claude-based actions via repository comments. Both workflows invoked the anthropics/claude-code-action@beta action with OAuth token authentication.

Changes

Cohort / File(s) Summary
GitHub Actions Workflows
.github/workflows/claude-code-review.yml, .github/workflows/claude.yml
Deleted two automated workflow files (141 lines total) that triggered Claude-based code reviews on PRs and issue/PR comments with @claude mentions.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Two workflows hop away,
No more code reviews in the fray,
Claude's actions fade to grey,
The workflows have their say.
👋✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is entirely empty of meaningful content, containing only the template placeholders with no actual details about changes, reasoning, or testing performed. Fill in the description with: (1) a summary of why Claude workflows are being removed, (2) select the appropriate 'Type of Change' checkbox, (3) add any related issue links, and (4) complete the testing and notes sections.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'remove claude from ci' is clear and directly related to the changeset, which removes two Claude-based GitHub Actions workflows from the CI configuration.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch marko/remove_claude

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 and usage tips.

@tac0turtle tac0turtle merged commit d08882e into main Apr 21, 2026
13 checks passed
@tac0turtle tac0turtle deleted the marko/remove_claude branch April 21, 2026 11:34
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