Skip to content

ci: claude-pr-review: Improve no comments text #3921

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
patrickelectric merged 1 commit into from
Apr 30, 2026
+2 −2
Merged

ci: claude-pr-review: Improve no comments text #3921

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/claude-pr-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ jobs:
- No findings at all → LGTM :shipit:
- If the verdict is not LGTM :shipit:, add a line listing the section numbers of every `critical`/`major` finding (e.g. "Critical items to address: 1.1, 3.2").
- One short paragraph (1–3 sentences) describing what the PR does at a high level.
- If (and only if) the verdict is LGTM :shipit:, add this exact line on its own after the high-level paragraph: Nothing to comment, nice job :thumbsup:
- If (and only if) the verdict is LGTM :shipit:, add this exact line on its own after the high-level paragraph: No further comments, nice job :thumbsup:

### 1. Correctness & Implementation Bugs
- Logic errors, off-by-ones, null/undefined hazards, race conditions, broken error handling, blocking calls in async code, incorrect MAVLink handling, wrong Vue 2 reactivity patterns (e.g. setting new keys without `Vue.set`), broken TypeScript or Python type hints, regressions.
Expand Down Expand Up @@ -222,7 +222,7 @@ jobs:

Renames `getCwd` to `getCurrentWorkingDirectory` across `core/services/helper/` and updates the two call sites in the frontend. No behavioral change.

Nothing to comment, nice job :thumbsup:
No further comments, nice job :thumbsup:

_Generated by PR Review Bot. This is advisory, a human reviewer must still approve._

Expand Down
Loading