From dea025510b6af86578bfeba58bdac7203c880a85 Mon Sep 17 00:00:00 2001 From: peaklabs-dev <122374094+peaklabs-dev@users.noreply.github.com> Date: Sun, 15 Feb 2026 23:53:11 +0100 Subject: [PATCH 1/2] ci: improve pr quality workflow - change labels to quality - add a failure label - make sure service json files are not changed --- .github/workflows/pr-quality.yaml | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/pr-quality.yaml b/.github/workflows/pr-quality.yaml index d199d0bba..d264ad470 100644 --- a/.github/workflows/pr-quality.yaml +++ b/.github/workflows/pr-quality.yaml @@ -56,7 +56,10 @@ jobs: SECURITY.md LICENSE CODE_OF_CONDUCT.md + templates/service-templates-latest.json + templates/service-templates.json require-final-newline: true + # User Health Checks min-repo-merged-prs: 0 min-repo-merge-ratio: 0 @@ -73,7 +76,7 @@ jobs: renovate[bot] github-actions[bot] exempt-draft-prs: false - exempt-label: "status/exempt" + exempt-label: "quality/exempt" exempt-pr-label: "" exempt-milestones: "" exempt-pr-milestones: "" @@ -81,7 +84,7 @@ jobs: exempt-all-pr-milestones: false # PR Success Actions - success-add-pr-labels: "status/verified" + success-add-pr-labels: "quality/verified" # PR Failure Actions close-pr: true @@ -90,4 +93,4 @@ jobs: failure-pr-message: "This PR did not pass quality checks so it will be closed. If you believe this is a mistake please let us know." failure-remove-pr-labels: "" failure-remove-all-pr-labels: true - failure-add-pr-labels: "" + failure-add-pr-labels: "quality/rejected" From 362fc770f1a6d2f6ae0e864b485ea7f7c0086b32 Mon Sep 17 00:00:00 2001 From: peaklabs-dev <122374094+peaklabs-dev@users.noreply.github.com> Date: Sun, 15 Feb 2026 23:54:01 +0100 Subject: [PATCH 2/2] ci: delete label removal workflow --- ...e-remove-labels-and-assignees-on-close.yml | 86 ------------------- 1 file changed, 86 deletions(-) delete mode 100644 .github/workflows/chore-remove-labels-and-assignees-on-close.yml diff --git a/.github/workflows/chore-remove-labels-and-assignees-on-close.yml b/.github/workflows/chore-remove-labels-and-assignees-on-close.yml deleted file mode 100644 index 8ac199a08..000000000 --- a/.github/workflows/chore-remove-labels-and-assignees-on-close.yml +++ /dev/null @@ -1,86 +0,0 @@ -name: Remove Labels and Assignees on Issue Close - -on: - issues: - types: [closed] - pull_request: - types: [closed] - pull_request_target: - types: [closed] - -permissions: - issues: write - pull-requests: write - -jobs: - remove-labels-and-assignees: - runs-on: ubuntu-latest - steps: - - name: Remove labels and assignees - uses: actions/github-script@v7 - with: - github-token: ${{ secrets.GITHUB_TOKEN }} - script: | - const { owner, repo } = context.repo; - - async function processIssue(issueNumber, isFromPR = false, prBaseBranch = null) { - try { - if (isFromPR && prBaseBranch !== 'v4.x') { - return; - } - - const { data: currentLabels } = await github.rest.issues.listLabelsOnIssue({ - owner, - repo, - issue_number: issueNumber - }); - - const labelsToKeep = currentLabels - .filter(label => label.name === '⏱︎ Stale') - .map(label => label.name); - - await github.rest.issues.setLabels({ - owner, - repo, - issue_number: issueNumber, - labels: labelsToKeep - }); - - const { data: issue } = await github.rest.issues.get({ - owner, - repo, - issue_number: issueNumber - }); - - if (issue.assignees && issue.assignees.length > 0) { - await github.rest.issues.removeAssignees({ - owner, - repo, - issue_number: issueNumber, - assignees: issue.assignees.map(assignee => assignee.login) - }); - } - } catch (error) { - if (error.status !== 404) { - console.error(`Error processing issue ${issueNumber}:`, error); - } - } - } - - if (context.eventName === 'issues') { - await processIssue(context.payload.issue.number); - } - - if (context.eventName === 'pull_request' || context.eventName === 'pull_request_target') { - const pr = context.payload.pull_request; - await processIssue(pr.number); - if (pr.merged && pr.base.ref === 'v4.x' && pr.body) { - const issueReferences = pr.body.match(/#(\d+)/g); - if (issueReferences) { - for (const reference of issueReferences) { - const issueNumber = parseInt(reference.substring(1)); - await processIssue(issueNumber, true, pr.base.ref); - } - } - } - }