Skip to content

test: intentional violations to test CI workflows#3

Closed
machado144 wants to merge 1 commit intomainfrom
test/trigger-violations
Closed

test: intentional violations to test CI workflows#3
machado144 wants to merge 1 commit intomainfrom
test/trigger-violations

Conversation

@machado144
Copy link
Contributor

Summary

  • Adds forbidden files (.env.local, debug.log, tmp/) to trigger structlint violations
  • Adds unused global variable to trigger reviewforge feedback
  • This PR is intentionally broken — used to verify both workflows catch issues

Expected Results

  • StructLint: should comment with violations (disallowed files + disallowed directory)
  • ReviewForge: should flag the unused globalDebug variable and TODO hack

Generated with Claude Code

Intentionally adding .env.local, debug.log, tmp/ directory, and an
unused global variable to verify both structlint and reviewforge
catch issues on PR.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Mar 7, 2026

❌ StructLint Validation

Metric Count
Checks passed 78
Violations 2
Violation Details

⚠️ Directories not in the allowed list (1)

  • Directory not in allowed list: tmp

⚠️ Files matching disallowed naming patterns (1)

  • Disallowed file naming pattern found: .env.local

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Core Changes

  • An unused global variable globalDebug has been added to internal/validator/validator.go along with a TODO comment.

Verdict

Approve: The change is minor and does not introduce any functional bugs, security vulnerabilities, breaking changes, or crashes. The PR description indicates this change is for testing CI workflows.


Code review performed by GEMINI - gemini-2.5-flash.

@machado144 machado144 closed this Mar 7, 2026
@machado144 machado144 deleted the test/trigger-violations branch March 7, 2026 21:13
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.

1 participant