From 5d7258570523c02f82ffab3fd77e9cfda5215a17 Mon Sep 17 00:00:00 2001 From: Alois Klink Date: Thu, 8 Sep 2022 03:51:42 +0100 Subject: [PATCH] ci: re-enable coveralls coverage upload Reverts commits: - Disabling coveralls temporarilly https://github.com/mermaid-js/mermaid/commit/aedf0663379dc4d03e898b39874fc0686bfd8697 - Tmo disabling of coveralls https://github.com/mermaid-js/mermaid/commit/a6b4cb24b8208f661bae838489059296a0ec07e4 I'm not 100% sure whether the `parallel` option is needed, but I've left it disabled for now, since it'd would mean that we need to merge both the `test.yml` and the `e2e.yml` file together. --- .github/workflows/e2e | 6 ++++++ .github/workflows/test.yml | 9 +++++++++ 2 files changed, 15 insertions(+) diff --git a/.github/workflows/e2e b/.github/workflows/e2e index 5b716e429..338869490 100644 --- a/.github/workflows/e2e +++ b/.github/workflows/e2e @@ -36,3 +36,9 @@ jobs: run: yarn e2e env: CYPRESS_CACHE_FOLDER: .cache/Cypress + + - name: Upload Coverage to Coveralls + uses: coverallsapp/github-action@master + with: + github-token: ${{ secrets.GITHUB_TOKEN }} + flag-name: e2e diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 97aa0a377..08c35befa 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -32,3 +32,12 @@ jobs: - name: Run Unit Tests run: | yarn ci --coverage + + - name: Upload Coverage to Coveralls + # it feels a bit weird to use @master, but that's what the docs use + # (coveralls also doesn't publish a @v1 we can use) + # https://github.com/marketplace/actions/coveralls-github-action + uses: coverallsapp/github-action@master + with: + github-token: ${{ secrets.GITHUB_TOKEN }} + flag-name: unit