From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id BE9E54CCC0 for ; Fri, 8 Aug 2025 20:49:16 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id C346768CD36; Fri, 8 Aug 2025 23:49:12 +0300 (EEST) Received: from 0f9ae49ae7c8 (code.ffmpeg.org [188.245.149.3]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id BA1A968B999 for ; Fri, 8 Aug 2025 23:49:11 +0300 (EEST) MIME-Version: 1.0 From: Timo Rothenpieler To: ffmpeg-devel@ffmpeg.org Subject: [FFmpeg-devel] =?utf-8?q?=5BPATCH=5D_forgejo/autolabeler=3A_add_?= =?utf-8?q?some_logic_to_also_auto-remove_the_=22New=22_label_=28PR_=23201?= =?utf-8?b?ODYp?= X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Message-Id: <20250808204912.C346768CD36@ffbox0-bg.ffmpeg.org> Date: Fri, 8 Aug 2025 23:49:12 +0300 (EEST) Archived-At: List-Archive: List-Post: PR #20186 opened by Timo Rothenpieler (BtbN) URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20186 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20186.patch >From b5ab096897e92a56642a17a6ef212620f5c7d79b Mon Sep 17 00:00:00 2001 From: Timo Rothenpieler Date: Fri, 8 Aug 2025 21:37:20 +0200 Subject: [PATCH] forgejo/autolabeler: add some logic to also auto-remove the "New" label --- .forgejo/labeler/labeler.js | 48 ++++++++++++++++++++++++++++---- .forgejo/workflows/autolabel.yml | 7 +++-- 2 files changed, 47 insertions(+), 8 deletions(-) diff --git a/.forgejo/labeler/labeler.js b/.forgejo/labeler/labeler.js index 7e17c4167c..b1c05787f0 100644 --- a/.forgejo/labeler/labeler.js +++ b/.forgejo/labeler/labeler.js @@ -1,6 +1,7 @@ module.exports = async ({github, context}) => { const title = (context.payload.pull_request?.title || context.payload.issue?.title || '').toLowerCase(); const labels = []; + const issueNumber = context.payload.pull_request?.number || context.payload.issue?.number; const kwmap = { 'avcodec': 'avcodec', @@ -13,15 +14,50 @@ module.exports = async ({github, context}) => { 'fftools': 'CLI' }; - if (context.payload.action === 'opened') { + async function isOrgMember(username) { + try { + const response = await github.rest.orgs.checkMembershipForUser({ + org: context.repo.owner, + username: username + }); + return response.status === 204; + } catch (error) { + return false; + } + } + + if (context.payload.action === 'closed' || + (context.payload.action !== 'opened' && ( + context.payload.action === 'assigned' || + context.payload.action === 'label_updated' || + context.payload.comment) && + await isOrgMember(context.payload.sender.login)) + ) { + try { + await github.rest.issues.removeLabel({ + owner: context.repo.owner, + repo: context.repo.repo, + issue_number: issueNumber, + // this should say 'new', but forgejo deviates from GitHub API here and expects the ID + name: '41' + }); + console.log('Removed "new" label'); + } catch (error) { + if (error.status !== 404 && error.status !== 410) { + console.log('Could not remove "new" label'); + } + } + } else if (context.payload.action === 'opened') { labels.push('new'); console.log('Detected label: new'); } - for (const [kw, label] of Object.entries(kwmap)) { - if (title.includes(kw)) { - labels.push(label); - console.log('Detected label: ' + label); + if ((context.payload.action === 'opened' || context.payload.action === 'edited') && context.eventName !== 'issue_comment') { + for (const [kw, label] of Object.entries(kwmap)) { + if (title.includes(kw)) { + labels.push(label); + console.log('Detected label: ' + label); + } } } @@ -29,7 +65,7 @@ module.exports = async ({github, context}) => { await github.rest.issues.addLabels({ owner: context.repo.owner, repo: context.repo.repo, - issue_number: context.payload.pull_request?.number || context.payload.issue?.number, + issue_number: issueNumber, labels: labels, }); } diff --git a/.forgejo/workflows/autolabel.yml b/.forgejo/workflows/autolabel.yml index 3eb8cd52ba..cb4dac67aa 100644 --- a/.forgejo/workflows/autolabel.yml +++ b/.forgejo/workflows/autolabel.yml @@ -1,12 +1,15 @@ on: pull_request_target: - types: [opened, edited, synchronize] + types: [opened, edited, synchronize, closed, assigned, labeled, unlabeled] issues: - types: [opened, edited] + types: [opened, edited, closed, assigned, labeled, unlabeled] + issue_comment: + types: [created] jobs: pr_labeler: runs-on: utilities + if: ${{ github.event.sender.login != 'ffmpeg-devel' }} steps: - name: Checkout uses: actions/checkout@v4 -- 2.49.1 _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".