Skip to content
This repository has been archived by the owner on Oct 11, 2024. It is now read-only.

Fix step-status script #333

Merged
merged 3 commits into from
Jun 27, 2024
Merged

Fix step-status script #333

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
27e3f40
Preserve comparison integrity
dbarbuzzi Jun 25, 2024
cf26778
Remove unused variable
dbarbuzzi Jun 25, 2024
2778c1e
Merge branch 'main' into fix-step-status
dbarbuzzi Jun 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions .github/actions/nm-summary-build/action.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@ runs:
using: composite
steps:
- run: |
BUILD_STATUS=${{ inputs.build_status }}
BUILD_EMOJI=$(./.github/scripts/step-status ${BUILD_STATUS})
WHL_STATUS=${{ inputs.whl_status }}
WHL_EMOJI=$(./.github/scripts/step-status ${WHL_STATUS})
echo "testmo URL: ${{ inputs.testmo_run_url }}" >> $GITHUB_STEP_SUMMARY
Expand Down
2 changes: 1 addition & 1 deletion .github/scripts/step-status
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

STEP_STATUS=${1}

if [ $STEP_STATUS -eq 0 ]; then
if [ "$STEP_STATUS" -eq 0 ]; then
# green check
echo -e "\xE2\x9C\x85"
else
Expand Down
Loading