Skip to content

chore(ci): remove Claude Code review workflow - #108

Merged
1 commit merged into
mainfrom
chore/remove-claude-code-review
May 6, 2026
Merged

chore(ci): remove Claude Code review workflow#108
1 commit merged into
mainfrom
chore/remove-claude-code-review

Conversation

@ANcpLua

@ANcpLua ANcpLua commented May 6, 2026

Copy link
Copy Markdown
Owner

Summary

Removes .github/workflows/claude-code-review.yml. The GitHub Claude app integration will be rebuilt from scratch — the existing wiring has been flaky.

Test plan

  • Required CI checks (build (ubuntu/windows), version) still run green
  • No remaining references to claude-code-review anywhere in the repo

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Internal workflow configuration adjusted; no observable impact on functionality or user experience.
    • No changes to public APIs or exported interfaces. No end-user facing changes in this release.

Copilot AI review requested due to automatic review settings May 6, 2026 05:20
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@ghost
ghost enabled auto-merge (squash) May 6, 2026 05:20
@coderabbitai

coderabbitai Bot commented May 6, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 3a7b8348-af3f-4690-bbd8-87f186851672

📥 Commits

Reviewing files that changed from the base of the PR and between 0f13cbc and fc12da3.

📒 Files selected for processing (1)
  • .github/workflows/claude-code-review.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/claude-code-review.yml
📜 Recent review details
⏰ Context from checks skipped due to timeout of 180000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: build (ubuntu-latest)
  • GitHub Check: build (windows-latest)

📝 Walkthrough

Walkthrough

The PR touches .github/workflows/claude-code-review.yml. The diff contains an empty new hunk and an old hunk with identical content, resulting in no net changes to the file. No exported/public signatures were modified; lines changed show +0/-63 with the final workflow unchanged.

🚥 Pre-merge checks | ✅ 8 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
No Null-Forgiving Operator Without Justification ⚠️ Warning Multiple null-forgiving operators (!) found without inline comments in newly added C# files including ClassMetadata.cs, FieldCodeGenerator.cs, PropertyCodeGenerator.cs, MethodCodeGenerator.cs. Add inline comments justifying all null-forgiving operators. Examples: ClassMetadata.cs:20 (null!), FieldCodeGenerator.cs:50,57,61, PropertyCodeGenerator.cs:46,53,57, MethodCodeGenerator.cs:123, and attribute metadata files.
✅ Passed checks (8 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Title follows conventional commits format (chore scope), is under 72 characters (45 chars), contains no trailing period, and accurately describes the removal of the Claude Code review workflow.
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.
No Datetime.Now/Utcnow ✅ Passed PR removes only a YAML workflow file (.github/workflows/claude-code-review.yml). No C# files added or modified. Custom check targets C# files only, therefore not applicable.
No .Result/.Wait() Blocking Async ✅ Passed Scanned 192 added C# files for blocking async patterns (.Result, .Wait(), .GetAwaiter().GetResult()). Zero occurrences found across src/ and tests/ directories. No modified C# files detected.
No Isourcegenerator ✅ Passed No ISourceGenerator implementations found. AotReflectionGenerator correctly implements IIncrementalGenerator, the modern standard. 192 added C# files scanned; none violate the check.
Sources Public Types Must Be Internal ✅ Passed No C# files added or modified in src/ANcpLua.Roslyn.Utilities.Sources/. The custom check scans C# files for public types; zero C# files means zero violations. Pass.

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


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

Removing claude-code-review.yml — the GitHub Claude app integration
will be rebuilt fresh; the existing wiring has been flaky.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ANcpLua
ANcpLua force-pushed the chore/remove-claude-code-review branch from 0f13cbc to fc12da3 Compare May 6, 2026 05:40
@github-actions

github-actions Bot commented May 6, 2026

Copy link
Copy Markdown

@coderabbitai autofix

@coderabbitai

coderabbitai Bot commented May 6, 2026

Copy link
Copy Markdown

Note

Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it.

Autofix skipped. No unresolved CodeRabbit review comments with fix instructions found.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ghost
ghost merged commit 1678d16 into main May 6, 2026
12 checks passed
@ghost
ghost deleted the chore/remove-claude-code-review branch May 6, 2026 05:42
This pull request was closed.
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