-
Notifications
You must be signed in to change notification settings - Fork 312
Closed as not planned
Closed as not planned
Copy link
Labels
Description
We looked at 5 open PRs — 4 look great and are ready for eyes, and 1 needs a small improvement before merging.
Ready to review 🟢
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #22113 | [architecture] Update architecture diagram scratchpad | @github-actions[bot] |
2 | lgtm ✨ |
| #22106 | Fix: Turns always 0 and ToolCalls always null in run summaries | @Copilot |
295 | lgtm ✨ |
| #22105 | Rename policyStatuses and getActivationOutputsCodemod for discoverability |
@Copilot |
34 | lgtm ✨ |
| #22005 | Preserve angle brackets in code blocks and inline code spans | @Copilot |
273 | lgtm ✨ |
Needs a closer look 🟡
| PR | Title | Author | Lines | Quality |
|---|---|---|---|---|
| #22104 | Pre-compile heredoc regexp patterns at package level | @Copilot |
47 | needs-work |
#22104: The performance optimization is clean and idiomatic, but the diff is missing benchmark coverage to substantiate the performance claim. A
BenchmarkRemoveHeredocContenttest would make the improvement concrete and guard against future regressions. A comment with a ready-to-use prompt has been posted on the PR.
Evaluated: 5 · Skipped: 0 · Run: #23376296988
Generated by Contribution Check · ◷
- expires on Mar 22, 2026, 9:08 AM UTC
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Type
Fields
Give feedbackNo fields configured for issues without a type.