From 66ba9e06d92542268c97c500d5312100c299d973 Mon Sep 17 00:00:00 2001 From: ethernet Date: Thu, 25 Jun 2026 12:35:13 -0400 Subject: [PATCH] change(ci): remove lint PR comment it's already in the job summary. having it as a comment just makes people ignore it. don't waste sapce. --- .github/workflows/lint.yml | 40 -------------------------------------- 1 file changed, 40 deletions(-) diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml index 511119ca6..fcee2c1b8 100644 --- a/.github/workflows/lint.yml +++ b/.github/workflows/lint.yml @@ -109,46 +109,6 @@ jobs: --output .lint-reports/summary.md cat .lint-reports/summary.md >> "$GITHUB_STEP_SUMMARY" - - name: Upload reports as artifact - uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7 - with: - name: lint-reports - path: .lint-reports/ - retention-days: 14 - - - name: Post / update PR comment - if: inputs.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository - continue-on-error: true - uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7 - with: - script: | - const fs = require('fs'); - const body = fs.readFileSync('.lint-reports/summary.md', 'utf8'); - const marker = ''; - const fullBody = marker + '\n' + body; - - const { data: comments } = await github.rest.issues.listComments({ - owner: context.repo.owner, - repo: context.repo.repo, - issue_number: context.issue.number, - }); - const existing = comments.find(c => c.body && c.body.includes(marker)); - if (existing) { - await github.rest.issues.updateComment({ - owner: context.repo.owner, - repo: context.repo.repo, - comment_id: existing.id, - body: fullBody, - }); - } else { - await github.rest.issues.createComment({ - owner: context.repo.owner, - repo: context.repo.repo, - issue_number: context.issue.number, - body: fullBody, - }); - } - ruff-blocking: # Enforce the rules in pyproject.toml [tool.ruff.lint.select]. Currently # PLW1514 (unspecified-encoding) — catches bare ``open()`` /