diff --git a/.forgejo/workflows/ci.yml b/.forgejo/workflows/ci.yml index 5b04bd9c6..57ef6794d 100644 --- a/.forgejo/workflows/ci.yml +++ b/.forgejo/workflows/ci.yml @@ -338,7 +338,9 @@ jobs: runs-on: docker container: image: ${{vars.docker_prefix}}python:3.13-slim - needs: [lint, typecheck, security, quality] + # unit_tests is included so coverage only runs after tests pass, + # preventing misleading results when tests are still in-flight or failing. + needs: [lint, typecheck, security, quality, unit_tests] steps: - name: Install system dependencies (nodejs for checkout, git for merge tests) run: | diff --git a/CHANGELOG.md b/CHANGELOG.md index 796db5544..abb8dd788 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,6 +7,13 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). ### Changed +- **CI coverage job now waits for unit_tests** (#10714): Added `unit_tests` to the + `needs` list of the `coverage` job in `ci.yml`. Previously the coverage job ran + in parallel with unit tests, which could produce misleading pass results when + tests were still in-flight or had already failed. Coverage now only starts after + unit tests succeed, eliminating redundant parallel test execution and ensuring + coverage results are always meaningful. + - **Diagnostics spec examples expanded to all 9 providers** (#5320): Updated the `agents diagnostics` command examples in the specification to show all 9 supported providers (OpenAI, Anthropic, Google, Gemini, Azure, OpenRouter, Cohere, Groq,