Merge pull request 'fix: codeberg_api_all does not accept a custom token parameter (#198)' (#226) from fix/issue-198 into main

This commit is contained in:
johba 2026-03-19 02:29:14 +01:00
commit e6eb9e76e0
2 changed files with 3 additions and 1 deletions

View file

@ -60,8 +60,10 @@ codeberg_api() {
# Paginate a Codeberg API GET endpoint and return all items as a merged JSON array.
# Usage: codeberg_api_all /path (no existing query params)
# codeberg_api_all /path?a=b (with existing params — appends &limit=50&page=N)
# codeberg_api_all /path TOKEN (optional second arg: token; defaults to $CODEBERG_TOKEN)
codeberg_api_all() {
local path_prefix="$1"
local CODEBERG_TOKEN="${2:-${CODEBERG_TOKEN}}"
local sep page page_items count all_items="[]"
case "$path_prefix" in
*"?"*) sep="&" ;;

View file

@ -759,7 +759,7 @@ if [ "${POST_CODE}" = "201" ]; then
REVIEW_BOT_LOGIN=$(printf '%s' "$REVIEW_BOT_RESP" | jq -r '.login // empty')
fi
if [ -n "$REVIEW_BOT_LOGIN" ]; then
ALL_PR_REVIEWS=$(codeberg_api_all "/pulls/${PR_NUMBER}/reviews" || echo "[]")
ALL_PR_REVIEWS=$(codeberg_api_all "/pulls/${PR_NUMBER}/reviews" "$REVIEW_BOT_TOKEN" || echo "[]")
while IFS= read -r review_id; do
DISMISS_CODE=$(curl -s -o /dev/null -w "%{http_code}" \
-X POST \