-
Notifications
You must be signed in to change notification settings - Fork 1
Fix CI build #725
Copy link
Copy link
Open
Labels
needs-reviewHuman decision requiredHuman decision requiredp1ImportantImportantsize:SSmall — under 1 hourSmall — under 1 hourstage:investigateAnalyzing — not coding yetAnalyzing — not coding yet
Description
Context
The CI pipeline currently runs 4 GitHub Actions workflows on push/PR:
build-and-test.yml— builds and runs the internal test suite (this is the one to keep)quickdup.yml— runs QuickDup code duplication analysis (requires Go setup)claude-code-review.yml— runs Claude-powered code review on PRsclaude.yml— runs Claude Code action on @claude mentions
The extra workflows add complexity without sufficient value at this stage. The internal test suite is the primary quality gate.
Success Criteria
- Only
build-and-test.ymlremains in.github/workflows/ -
quickdup.yml,claude-code-review.yml, andclaude.ymlare deleted - CI runs successfully on a PR with only the build-and-test workflow
- No other CI configuration files are affected
Rules
- Do not modify
build-and-test.yml— it is already correct - Delete the other 3 workflow files entirely
- Do not remove any repo-level GitHub settings (branch protection, etc.) — only in-repo workflow files
NFRs
- CI should complete in under 5 minutes
Quality Gates
- Tests pass on the PR itself using the simplified CI
- No regressions in build-and-test workflow behavior
Faktoria Attribution: agent=intake | trigger=new issue intake
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
needs-reviewHuman decision requiredHuman decision requiredp1ImportantImportantsize:SSmall — under 1 hourSmall — under 1 hourstage:investigateAnalyzing — not coding yetAnalyzing — not coding yet