Paginate label lookup #1

Merged
stephen merged 2 commits from fix-label-pagination into main 2026-06-08 05:16:42 +00:00
Owner

Label lookup only read the first repository label page. In repos with more than one page of labels, sync-labels could miss an existing configured label and fail to remove it correctly.

This uses the existing paginator for label lookup and adds a mock Forgejo regression that places the target label on page two.

Tested with make test.

Label lookup only read the first repository label page. In repos with more than one page of labels, sync-labels could miss an existing configured label and fail to remove it correctly. This uses the existing paginator for label lookup and adds a mock Forgejo regression that places the target label on page two. Tested with `make test`.
Paginate label lookup
Some checks failed
test / unit (pull_request) Failing after 21s
55b89c1342
Allow event PR numbers in labeler tests
All checks were successful
test / unit (pull_request) Successful in 6s
6d7b5f64a2
stephen referenced this pull request from a commit 2026-06-08 05:16:42 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rasterstate/forgejo-labeler-action!1
No description provided.