fix: evaluation report link + ADR-006 Code Review Tooling#345
Merged
rdmueller merged 2 commits intoMar 24, 2026
Conversation
Evaluates CodeRabbit Free vs. GitHub Copilot Review vs. Claude Code CLI vs. Claude Code App. Pugh matrix to be filled after parallel evaluation on next PR. Motivation: CodeRabbit Free Tier limit reached regularly. References LLM-Coding#164 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Fix relative link to evaluation-report.html (SPA serves from root) - Add ADR-006: evaluates CodeRabbit vs Copilot vs Claude Code for code reviews. Pugh matrix to be filled after parallel evaluation. References LLM-Coding#164 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
WalkthroughDer PR enthält zwei Dokumentationsänderungen: Korrektur eines relativen Link-Pfads in der Anchor-Evaluations-Datei durch Entfernen eines übergeordneten Verzeichnispräfix sowie Hinzufügen eines neuen Architecture Decision Record (ADR) zur Bewertung von Code-Review-Tools. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/anchor-evaluations.adoc`:
- Line 15: Die Link-Ziel-Angabe in der AsciiDoc-Zeile
link:evaluation-report.html[**View the latest evaluation results →**] ist
relativ falsch und führt im gebauten Site-Pfad auf docs/evaluation-report.html
statt auf die Root-Datei; ändere das Ziel zurück auf ../evaluation-report.html
(also wieder "../" vor den Dateinamen) damit der Link korrekt auf
/evaluation-report.html zeigt.
In `@docs/specs/adrs/adr-006-code-review-tooling.adoc`:
- Line 7: Replace Markdown-style double-asterisk bold markers with AsciiDoc
single-asterisk bold markers: change instances like "**Proposed**" to
"*Proposed*", and similarly convert all "**...**" occurrences in the file
(including the ranges noted around lines 16-18, 54-59, 103-104) to "*...*"; also
ensure bullet lists use single asterisks (`* item`) rather than Markdown dashes
or double-asterisk formats so the document follows the AsciiDoc convention.
- Line 51: The decision text "Der nächste inhaltliche PR wird parallel von allen
drei Tools reviewed." contradicts the ADR's matrix which lists four alternatives
(including Claude App); update the sentence to reflect the correct number of
tools (e.g., "allen vier Tools") or remove/adjust references so both the
decision paragraph and the evaluation matrix consistently state the same count;
apply the same correction at the other occurrence around line 65 and verify the
matrix/alternative headings (including "Claude App") and any summary text match
the chosen count.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro
Run ID: f4a552ae-4678-4965-bdca-7220ae3750e6
📒 Files selected for processing (2)
docs/anchor-evaluations.adocdocs/specs/adrs/adr-006-code-review-tooling.adoc
| Without systematic evaluation, our catalog is a collection of untested assumptions. | ||
|
|
||
| link:../evaluation-report.html[**View the latest evaluation results →**] | ||
| link:evaluation-report.html[**View the latest evaluation results →**] |
There was a problem hiding this comment.
Linkziel ist relativ falsch und bricht im generierten Website-Pfad.
Der neue Pfad zeigt im Build auf docs/evaluation-report.html, die Datei liegt aber auf Root-Level (/evaluation-report.html). Bitte ../ wiederherstellen.
🔧 Vorschlag
-link:evaluation-report.html[**View the latest evaluation results →**]
+link:../evaluation-report.html[**View the latest evaluation results →**]🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/anchor-evaluations.adoc` at line 15, Die Link-Ziel-Angabe in der
AsciiDoc-Zeile link:evaluation-report.html[**View the latest evaluation results
→**] ist relativ falsch und führt im gebauten Site-Pfad auf
docs/evaluation-report.html statt auf die Root-Datei; ändere das Ziel zurück auf
../evaluation-report.html (also wieder "../" vor den Dateinamen) damit der Link
korrekt auf /evaluation-report.html zeigt.
|
|
||
| == Status | ||
|
|
||
| **Proposed** - 2026-03-24 |
There was a problem hiding this comment.
Bitte AsciiDoc-Bold statt Markdown-Bold verwenden.
In .adoc sollten Hervorhebungen mit *text* geschrieben werden, nicht mit **text**.
As per coding guidelines: “Use bullet lists with * … and use *bold* for bold text (not **bold**)”.
Also applies to: 16-18, 54-59, 103-104
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/specs/adrs/adr-006-code-review-tooling.adoc` at line 7, Replace
Markdown-style double-asterisk bold markers with AsciiDoc single-asterisk bold
markers: change instances like "**Proposed**" to "*Proposed*", and similarly
convert all "**...**" occurrences in the file (including the ranges noted around
lines 16-18, 54-59, 103-104) to "*...*"; also ensure bullet lists use single
asterisks (`* item`) rather than Markdown dashes or double-asterisk formats so
the document follows the AsciiDoc convention.
|
|
||
| _Noch nicht entschieden — Evaluation läuft._ | ||
|
|
||
| Der nächste inhaltliche PR wird parallel von allen drei Tools reviewed. |
There was a problem hiding this comment.
Entscheidungstext und Matrix widersprechen sich bei der Anzahl der Tools.
In Line 51 steht „allen drei Tools“, die ADR vergleicht aber vier Alternativen (inkl. Claude App). Bitte auf eine konsistente Anzahl bringen, sonst ist der Evaluationsumfang unklar.
Also applies to: 65-65
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@docs/specs/adrs/adr-006-code-review-tooling.adoc` at line 51, The decision
text "Der nächste inhaltliche PR wird parallel von allen drei Tools reviewed."
contradicts the ADR's matrix which lists four alternatives (including Claude
App); update the sentence to reflect the correct number of tools (e.g., "allen
vier Tools") or remove/adjust references so both the decision paragraph and the
evaluation matrix consistently state the same count; apply the same correction
at the other occurrence around line 65 and verify the matrix/alternative
headings (including "Claude App") and any summary text match the chosen count.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
../evaluation-report.html→evaluation-report.html)References #164
Test plan
🤖 Generated with Claude Code
Summary by CodeRabbit
Dokumentation