Skip to content

Commit aac379f

Browse files
jathaydeclaude
andcommitted
chore(audit): drop unused ignore_paths method
Refactoring `ignored?` to inline both implicit and user ignore logic left `ignore_paths` orphaned — nothing in the gem or tests calls it. Per Copilot review on PR #4. Removing it (rather than wiring it back) to keep the actual filter behavior in one place; if a future use needs the combined list, reach for `IMPLICIT_IGNORE + Array(@config[ "ignore"])` directly. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent ff0efd3 commit aac379f

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

lib/guardrails/audit.rb

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -90,12 +90,6 @@ def scan_paths
9090
Array(paths)
9191
end
9292

93-
def ignore_paths
94-
# Returned for the audit config knob; the actual filter logic lives
95-
# in `ignored?` because implicit and user ignores match differently.
96-
IMPLICIT_IGNORE + Array(@config["ignore"] || [])
97-
end
98-
9993
def collect_files
10094
patterns = scan_paths.map { |p| File.join(p, "**/*.html.erb") }
10195
patterns

0 commit comments

Comments
 (0)