-
Notifications
You must be signed in to change notification settings - Fork 295
Closed as not planned
Labels
Description
We looked at 4 open PRs — 3 look great and are ready for review, and 1 needs a small addition before it lands.
Ready to review 🟢
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20631 | feat: support custom OpenAI and Anthropic API targets in AWF | @Claude |
390 | lgtm ✨ |
| #20630 | [jsweep] Clean messages_core.cjs | @github-actions[bot] |
198 | lgtm ✨ |
| #20143 | Replace hardcoded /opt/gh-aw/ paths with GH_AW_HOME |
@Copilot |
27,702 | lgtm ✨ |
⚠️ Note: PR #20143 has a merge conflict (dirty) that needs resolving before it can land.
Needs a closer look 🟡
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20639 | refactor: split safe_outputs_config.go into focused modules | @Copilot |
974 | needs-work |
A pure refactor with no test files in the diff. Adding minimal import/compile tests for the two new modules (following the existing
safe_outputs_*_import_test.gopattern) would give reviewers confidence the module boundaries are sound.
Evaluated: 4 · Skipped: 0 · Run: #22994436703
Generated by Contribution Check · ◷
- expires on Mar 13, 2026, 9:16 AM UTC
Reactions are currently unavailable