From 3aa5111237f09f4340bd411bf569f34ebcd8e95e Mon Sep 17 00:00:00 2001
From: CleverThis <hal9000@cleverthis.com>
Date: Tue, 28 Apr 2026 06:57:26 +0000
Subject: [PATCH] fix(ci): add unit_tests to coverage job needs to prevent
 misleading parallel results

The coverage job in ci.yml previously only depended on [lint, typecheck,
security, quality], allowing it to run in parallel with unit_tests. This
meant coverage could report a pass even when unit tests were still running
or had already failed, wasting CI resources and producing misleading results.

Adding unit_tests to the coverage job's needs list ensures coverage only
starts after tests have passed, eliminating the redundant parallel test
execution and making coverage results meaningful.

ISSUES CLOSED: #10714