Repository navigation
Fix clean-main unit test failures - #65864
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Existing concurrency documentation still incorrectly states that queue values must be literals.
Review effort: Balanced
Findings: 1
What changed in this PR
Fixes clean-main unit test failures by isolating network-dependent tests and aligning validation specifications.
Changes:
- Stubs rate-limit calls in the timeout test.
- Synchronizes scanner image documentation and tests.
- Permits expressions in concurrency queue schema validation.
| File | Description |
|---|---|
pkg/workflow/schemas/github-workflow.json |
Allows queue expressions. |
pkg/cli/spec_test.go |
Updates expected scanner images. |
pkg/cli/README.md |
Documents digest-pinned images. |
pkg/cli/context_cancellation_test.go |
Removes live rate-limit dependencies. |
|
@copilot address the following outstanding work in one pass:
Push the necessary fixes, reply to each listed review thread and resolve it when addressed. Ignore feedback already answered or resolved. Use the pr-finisher skill and stop when only human review or CI remains; do not trigger CI. Sous-chef head: 8841db5
|
Co-authored-by: gh-aw-bot <259018956+gh-aw-bot@users.noreply.github.com>
|
🎉 This pull request is included in a new release. Release: |

Summary
Validation
make agent-report-progress