separated coveralls:report into its own step

.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 }}