Skip to content

Commit 57fda2d

Browse files
🐛 Explictly tell action to exit on error with +e (#7)
<!-- Copyright (C) 2020-2024 Arm Limited or its affiliates and Contributors. All rights reserved. SPDX-License-Identifier: Apache-2.0 --> ### Description <!-- Please add any detail or context that would be useful to a reviewer. --> Explictly tell action to exit on error with +e ### Test Coverage <!-- Please put an `x` in the correct box e.g. `[x]` to indicate the testing coverage of this change. --> - [ ] This change is covered by existing or additional automated tests. - [ ] Manual testing has been performed (and evidence provided) as automated testing was not feasible. - [x] Additional tests are not required for this change (e.g. documentation update).
1 parent fd29be1 commit 57fda2d

File tree

2 files changed

+2
-6
lines changed

2 files changed

+2
-6
lines changed

action.yml

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,13 +36,8 @@ runs:
3636
sudo apt-get clean
3737
- name: Verify project
3838
run: |
39+
set +e
3940
verify-cmsis-example-sdk verify --project-dir ${{ inputs.project-directory }} --project-path ${{ inputs.project-file }} --out-dir out --api-key ${{ inputs.API_TOKEN }} --log-error 2
40-
status=$?
41-
echo "verify-cmsis-example-sdk exited with: $status"
42-
if [ $status -ne 0 ]; then
43-
echo "::error::Verification of CMSIS project failed"
44-
exit 1
45-
fi
4641
shell: bash
4742
- name: Upload any artifacts
4843
uses: actions/upload-artifact@v4

changes/20240704101751.bugfix

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
:bug: Explictly tell action to exit on error with +e

0 commit comments

Comments
 (0)