From 4d537009c6d2bf00f7177571909f61debff796c4 Mon Sep 17 00:00:00 2001 From: Wolfgang Walther Date: Sun, 15 Jun 2025 11:08:48 +0200 Subject: [PATCH] workflow/labels: save an API request With the previous commit we now have the `before` labels available already, which allows some simplification. --- .github/workflows/labels.yml | 18 ++++++------------ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/.github/workflows/labels.yml b/.github/workflows/labels.yml index f15d8b4a6dcb..2b487dc00407 100644 --- a/.github/workflows/labels.yml +++ b/.github/workflows/labels.yml @@ -140,16 +140,9 @@ jobs: expectedHash: artifact.digest }) - // Shortcut for all issue endpoints related to labels - const pr = { - ...context.repo, - issue_number: pull_request.number - } - // Get all currently set labels that we manage const before = - (await github.paginate(github.rest.issues.listLabelsOnIssue, pr)) - .map(({ name }) => name) + pull_request.labels.map(({ name }) => name) .filter(name => name.startsWith('10.rebuild') || name == '11.by: package-maintainer' || @@ -159,8 +152,7 @@ jobs: const approvals = new Set( (await github.paginate(github.rest.pulls.listReviews, { - owner: context.repo.owner, - repo: context.repo.repo, + ...context.repo, pull_number: pull_request.number })) .filter(review => review.state == 'APPROVED') @@ -180,7 +172,8 @@ jobs: await Promise.all( before.filter(name => !after.includes(name)) .map(name => github.rest.issues.removeLabel({ - ...pr, + ...context.repo, + issue_number: pull_request.number name })) ) @@ -189,7 +182,8 @@ jobs: const added = after.filter(name => !before.includes(name)) if (added.length > 0) { await github.rest.issues.addLabels({ - ...pr, + ...context.repo, + issue_number: pull_request.number labels: added }) }