-
Notifications
You must be signed in to change notification settings - Fork 295
Closed as not planned
Labels
Description
We looked at 3 recent open PRs — 2 look great and are ready for review, and 1 needs some focus work before it's ready.
Ready to review 🟢
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #21032 | [docs] Update dictation skill glossary with 20 new terms | @github-actions[bot] |
20 | lgtm ✨ |
| #21018 | fix: custom API endpoints for Codex/Claude/Gemini engines | @Copilot |
394 | lgtm ✨ |
Both PRs are well-scoped and fully aligned with the contribution guidelines. PR #21018 is still in draft state — flipping it to ready-for-review will unblock the merge queue.
Needs a closer look 🟡
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #20864 | refactor: replace hardcoded /opt/gh-aw/ paths with GhAwHome constants | @Copilot |
27,641 | needs-work |
The core work is solid, but the PR bundles 5 independent changesets (path-constants refactor, GHE CLI support, GHES PR fix, HTTP timeout changes, mcpg version bump). Splitting into focused PRs will make the large recompile diff much easier to review safely. A detailed comment with a ready-to-use split prompt has been posted to the PR.
Evaluated: 3 · Skipped: 0 · Run: 23107246786
Generated by Contribution Check · ◷
- expires on Mar 16, 2026, 9:11 AM UTC
Reactions are currently unavailable