From c4438af64656b2cc52ef898a0c5f5edd5f6c0db3 Mon Sep 17 00:00:00 2001
From: Tom Longhurst <30480171+thomhurst@users.noreply.github.com>
Date: Thu, 27 Jun 2024 10:21:49 +0100
Subject: [PATCH] Fix

---
 .../Modules/CodacyCodeCoverageUploader.cs                      | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/ModularPipelines.Build/Modules/CodacyCodeCoverageUploader.cs b/src/ModularPipelines.Build/Modules/CodacyCodeCoverageUploader.cs
index 3cf484699f..7cbcec7872 100644
--- a/src/ModularPipelines.Build/Modules/CodacyCodeCoverageUploader.cs
+++ b/src/ModularPipelines.Build/Modules/CodacyCodeCoverageUploader.cs
@@ -3,6 +3,7 @@
 using ModularPipelines.Build.Attributes;
 using ModularPipelines.Build.Settings;
 using ModularPipelines.Context;
+using ModularPipelines.Enums;
 using ModularPipelines.GitHub.Attributes;
 using ModularPipelines.Models;
 using ModularPipelines.Modules;
@@ -28,7 +29,7 @@ protected override async Task<SkipDecision> ShouldSkip(IPipelineContext context)
     {
         var mergeCoverageModuleResult = await GetModule<MergeCoverageModule>();
 
-        if (mergeCoverageModuleResult.Value == null)
+        if (mergeCoverageModuleResult.ModuleStatus != Status.Successful)
         {
             return "Merge coverage did not run";
         }