fix(dependency_updater): prevent infinite pagination loop by checking NextPage before advancing
The previous pagination logic placed the resp.NextPage == 0 check in an unreachable else if branch competing with TagPrefix conditions. As a result, when TagPrefix != "" and no matching release is found on the last page, the code set options.Page = 0 and could loop indefinitely.
This change:
- Moves the
resp.NextPage == 0termination check to run before assigningoptions.Page = resp.NextPage. - Removes the unreachable
else if resp.NextPage == 0branch.
The new flow follows the official go-github pagination example pattern, ensuring the loop exits correctly and preventing infinite iteration.
🟡 Heimdall Review Status
| Requirement | Status | More Info | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Reviews |
🟡
0/1
|
Denominator calculation
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This pull request was closed because it has been inactive for 5 days since being marked as stale.