Skip to content

Commit

Permalink
Avoid skipping the unittest for proper coverage generation
Browse files Browse the repository at this point in the history
Signed-off-by: Tyler Gu <jiaweig3@illinois.edu>
  • Loading branch information
tylergu committed Dec 15, 2023
1 parent 008cdcc commit cf22855
Showing 1 changed file with 0 additions and 14 deletions.
14 changes: 0 additions & 14 deletions .github/workflows/unittest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,19 +5,8 @@ on:
pull_request:
branches: [ main ]
jobs:
check_should_skip:
runs-on: ubuntu-latest
outputs:
should_skip: ${{ steps.skip_check.outputs.should_skip }}
steps:
- id: skip_check
uses: fkirc/skip-duplicate-actions@v5.3.1
with:
skip_after_successful_duplicate: 'true'
unittest:
runs-on: ubuntu-latest
needs: check_should_skip
if: ${{ needs.check_should_skip.outputs.should_skip != 'true' }}
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
Expand Down Expand Up @@ -52,8 +41,6 @@ jobs:
retention-days: 1
integration-test:
runs-on: ubuntu-latest
needs: check_should_skip
if: ${{ needs.check_should_skip.outputs.should_skip != 'true' }}
steps:
- uses: actions/checkout@v3
- uses: actions/setup-python@v3
Expand Down Expand Up @@ -89,7 +76,6 @@ jobs:
coverage-report:
runs-on: ubuntu-latest
needs: [unittest, integration-test]
if: always()
permissions:
pull-requests: write
contents: write
Expand Down

0 comments on commit cf22855

Please sign in to comment.