Skip to content

Commit

Permalink
WIP: refacto the ifs
Browse files Browse the repository at this point in the history
  • Loading branch information
soonum committed May 16, 2024
1 parent 328093f commit b6b130b
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/aws_tfhe_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ env:
SLACK_ICON: https://pbs.twimg.com/profile_images/1274014582265298945/OjBKP9kn_400x400.png
SLACK_USERNAME: ${{ secrets.BOT_USERNAME }}
SLACK_WEBHOOK: ${{ secrets.SLACK_WEBHOOK }}
IS_PULL_REQUEST: ${{ github.event_name == 'pull_request' }}

on:
# Allows you to run this workflow manually from the Actions tab as an alternative.
Expand All @@ -21,13 +22,13 @@ on:

jobs:
file-change:
if: ${{ github.event_name == 'pull_request' }}
# if: ${{ github.event_name == 'pull_request' }}
runs-on: ubuntu-latest
permissions:
pull-requests: write
outputs:
core_crypto_test: ${{ steps.changed-files.outputs.core_crypto_any_changed }}
boolean_test: ${{ steps.changed-files.outputs.boolean_any_changed }}
core_crypto_test: ${{ !env.IS_PULL_REQUEST || steps.changed-files.outputs.core_crypto_any_changed }}
boolean_test: ${{ !env.IS_PULL_REQUEST || steps.changed-files.outputs.boolean_any_changed }}
shortint_test: ${{ steps.changed-files.outputs.shortint_any_changed }}
c_api_test: ${{ steps.changed-files.outputs.c_api_any_changed }}
high_level_api_test: ${{ steps.changed-files.outputs.high_level_api_any_changed }}
Expand Down Expand Up @@ -109,7 +110,7 @@ jobs:
cpu-tests:
name: CPU tests
if: ((always() && github.event_name != 'pull_request') ||
(github.event_name == 'pull_request' && needs.setup-instance.outputs.runner-name != ''))
(github.event_name == 'pull_request' && needs.setup-instance.result != 'skipped'))
needs: [ file-change, setup-instance ]
concurrency:
group: ${{ github.workflow }}_${{ github.ref }}
Expand Down

0 comments on commit b6b130b

Please sign in to comment.