Skip to content

Commit

Permalink
Fix GH API path to include GH user
Browse files Browse the repository at this point in the history
The API call seems to miss the GH user and returns always `null` for me when running in terminal. 

Note: I have no idea what that `:-` does in line 34 + 35, can you enlighten me? 

For the API call on GH it should use the original GH user + repo name otherwise the to lowercased values (for DockerHub) will fail in this case? 

Maybe this fixes meeDamian#2 also. I had `description: true` and actually a description on my repo but it failed.
  • Loading branch information
ndeet authored Nov 18, 2019
1 parent 06fa190 commit 507e114
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ if [ -n "${INPUT_DESCRIPTION}" ]; then
# If it was set to `true`, then fetch from Github.
# No quoting necessary, as it either returns a valid JSON string, or null
if [ "${INPUT_DESCRIPTION}" = "true" ]; then
DESC=$(curl -s "https://api.github.com/repos/${SLUG}" | jq '.description')
DESC=$(curl -s "https://api.github.com/repos/${GITHUB_ACTOR}/${GITHUB_REPOSITORY}" | jq '.description')
fi
fi

Expand Down

0 comments on commit 507e114

Please sign in to comment.