Skip to content

Commit

Permalink
Update & fix Testmo actions/logic (#230)
Browse files Browse the repository at this point in the history
This PR updates and fixes Testmo actions/usage:

* All Testmo actions are updated to the latest version (which, among
other things, eliminates the need to install the CLI tool separately)
* Fixes a "bug" in the conditional logic so that it is properly
evaluated (it seems that GitHub doesn’t like mixing
functions / expressions in and out of the double-curly braces in these
places)
  • Loading branch information
dbarbuzzi authored Dec 23, 2024
1 parent 85b473e commit fe4a442
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 17 deletions.
10 changes: 3 additions & 7 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -76,13 +76,9 @@ jobs:
with:
ref: ${{ inputs.gitref }}

- name: install testmo
uses: neuralmagic/nm-actions/actions/install-testmo@v1.0.0

- name: create testmo run
id: create_testmo_run
uses: neuralmagic/nm-actions/actions/testmo-run-create@v1.2.0
if: success()
uses: neuralmagic/nm-actions/actions/testmo-run-create@v1.11.0
with:
testmo_url: https://neuralmagic.testmo.net
testmo_token: ${{ secrets.TESTMO_TEST_TOKEN }}
Expand Down Expand Up @@ -142,8 +138,8 @@ jobs:

- name: report build status to testmo
id: report_build
uses: neuralmagic/nm-actions/actions/testmo-run-submit-thread@v1.2.0
if: (success() || failure()) && ${{ inputs.testmo_run_id != '' }}
uses: neuralmagic/nm-actions/actions/testmo-run-submit-thread@v1.11.0
if: success() || failure()
with:
testmo_url: https://neuralmagic.testmo.net
testmo_token: ${{ secrets.TESTMO_TEST_TOKEN }}
Expand Down
7 changes: 2 additions & 5 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -90,9 +90,6 @@ jobs:
with:
venv: TEST

- name: install testmo
uses: neuralmagic/nm-actions/actions/install-testmo@v1.0.0

- name: download whl
id: download
uses: actions/download-artifact@v4
Expand All @@ -108,8 +105,8 @@ jobs:

- name: report test results
id: report_test
uses: neuralmagic/nm-actions/actions/testmo-run-submit-thread@v1.2.0
if: (success() || failure()) && ${{ inputs.testmo_run_id != '' }}
uses: neuralmagic/nm-actions/actions/testmo-run-submit-thread@v1.11.0
if: (success() || failure()) && inputs.testmo_run_id != ''
with:
testmo_url: https://neuralmagic.testmo.net
testmo_token: ${{ secrets.TESTMO_TEST_TOKEN }}
Expand Down
7 changes: 2 additions & 5 deletions .github/workflows/upload.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,12 +69,9 @@ jobs:
with:
python-version: 3.10.12

- name: install testmo
uses: neuralmagic/nm-actions/actions/install-testmo@v1.0.0

- name: complete testmo run
uses: neuralmagic/nm-actions/actions/testmo-run-complete@v1.2.0
if: (success() || failure()) && ${{ inputs.testmo_run_id != '' }}
uses: neuralmagic/nm-actions/actions/testmo-run-complete@v1.11.0
if: (success() || failure()) && inputs.testmo_run_id != ''
with:
testmo_url: https://neuralmagic.testmo.net
testmo_token: ${{ secrets.TESTMO_TEST_TOKEN }}
Expand Down

0 comments on commit fe4a442

Please sign in to comment.