-
Notifications
You must be signed in to change notification settings - Fork 288
Closed as not planned
Labels
Description
We looked at 2 new PRs β 1 looks great and is ready for review, and 1 doesn't fit the contribution guidelines.
Ready to review π’
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20272 | Add missing flags to run, audit, new, and l... | @Copilot |
116 | lgtm β¨ |
Details for #20272:
- β On-topic: Addresses CLI consistency issues from [cli-consistency] CLI Consistency Issues and Missing FlagsΒ #20218
- β Focused: Each change adds one missing flag per command
- β
Tests: Updated
commands_test.goappropriately - π‘ Note: JSON output behavior for
runcommand may need maintainer verification for partial failures
Off-guidelines π΄
Per-PR Details
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20143 | Replace hardcoded /opt/gh-aw/ paths with ... |
@Copilot |
24,913 | spam |
Issues with #20143:
- π« Unauthorized direct PR without approved agentic plan
- β Missing tests despite modifying 247 files including core compiler logic
- β No discussion issue for why PR feat: portable GH_AW_HOME for self-hosted runner supportΒ #19744 reimplementation was needed
Feedback posted to the PR with specific remediation steps.
Evaluated: 2 Β· Skipped: 5 Β· Run #22882715379
Generated by Contribution Check Β· β·
- expires on Mar 11, 2026, 1:31 AM UTC
Reactions are currently unavailable