From f4ad6e889d256744edd18656271d7772a82fab24 Mon Sep 17 00:00:00 2001
From: Phred <fearphage@gmail.com>
Date: Thu, 7 Oct 2021 16:35:34 -0500
Subject: [PATCH] separated coveralls:report into its own step

---
 .github/workflows/ci.yml | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 8fd49f0675b..1e10d13d744 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -42,6 +42,14 @@ jobs:
       # running in parallel causes some tests to fail
       - run: mvn test -B # -T 1C
 
+      # do we need all these goals? Most other projects just do jacoco:report and coveralls:report
+      - run: >
+          mvn
+          jacoco:report
+          jacoco:report-aggregate
+          verify
+          --define jacoco.skip=false
+
       - env:
           CI_BUILD_NUMBER: ${{ github.run_id }}
           # this will need to be updated to work on the main branch
@@ -51,9 +59,5 @@ jobs:
           CI_PULL_REQUEST: ${{ github.event.pull_request.number }}
         run: >
           mvn
-          jacoco:report
-          jacoco:report-aggregate
-          verify
           coveralls:report
-          --define jacoco.skip=false
           --define repoToken=${{ secrets.GITHUB_TOKEN }}