Skip to content

Commit 8384e86

Browse files
committed
Refinements
1 parent f37cf93 commit 8384e86

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

.claude/skills/check-code-attribution/SKILL.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ If the diff deletes a file and the removed lines contained attribution headers,
156156
| **medium** | ⚠️ Missing attribution header fields, stripped headers, missing/inconsistent NOTICES entries, deleted/renamed vendored files needing NOTICES update |
157157
| **low** | 👀 Attribution present but could be improved |
158158

159-
Note: Warden relies on these severity levels when deciding whether to make comments on PRs or require changes.
159+
Note: Note: Warden relies on these severity levels when deciding whether to make comments on PRs or require changes. Always include the emoji icon (🚨, ⚠️, 👀) in each finding so reviewers can triage at a glance.
160160

161161
## Output
162162

warden.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,8 +80,8 @@ ignorePaths = [
8080
[[skills.triggers]]
8181
type = "pull_request"
8282
actions = ["opened", "synchronize"]
83-
requestChanges = true # Post a blocking "Request Changes" PR comment when failOn-level findings exist.
84-
failCheck = true # Red X on task in GitHub when failOn-level findings exist (currently non-blocking).
83+
requestChanges = false # Non-blocking COMMENT reviews only — no "Request Changes" PR reviews.
84+
failCheck = false # Advisory only — do not fail the GitHub check.
8585

8686
[[skills.triggers]]
8787
type = "local"

0 commit comments

Comments
 (0)