Skip to content

Commit

Permalink
fix: assets upload to use the correct variable name.
Browse files Browse the repository at this point in the history
Added needs on for the upload job to get the upload url from the create
release job.

Signed-off-by: yash-zededa <yash@zededa.com>
  • Loading branch information
yash-zededa committed Oct 31, 2024
1 parent 7534022 commit d38d007
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/assets.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,12 @@ jobs:
}' https://api.github.com/repos/${{ github.repository }}/releases)
release_id=$(echo "$response" | jq -r .id)
upload_url=$(echo "$response" | jq -r .upload_url | sed -e "s/{?name,label}//")
echo $upload_url
echo "release_id=$release_id" >> "$GITHUB_OUTPUT"
echo "upload_url=$upload_url" >> "$GITHUB_OUTPUT"
build:
runs-on: ubuntu-20.04
needs: create_release
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -117,7 +119,7 @@ jobs:
# Add ARCH prefix
new_name="${ARCH}.${base_name}"
# Rename the file
mv "$asset" "assets/$new_name"
mv "$file" "assets/$new_name"
echo "Uploading assets/$new_name as $new_name..."
upload_response=$(curl -s -X POST \
-H "Authorization: Bearer $GITHUB_TOKEN" \
Expand Down

0 comments on commit d38d007

Please sign in to comment.