Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

fix(ci): signed-commit check fails pr-refine on PRs over 250 commits even when the lane made no commits

Đang mở
  • Có gì không ổn?
#7,256 1 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
57/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, shell
Lĩnh vực
ci-cd

Hướng nghiên cứu

Start with .github/actions/check-signed-commits/check-signed-commits.mjs, especially the 250-commit cap, completeness check, and escalation label request. Trace how the action identifies commits made by the lane and how it handles a lane with no commits; use the observed run on #7009 as a reference. Done means the check covers only the lane's commits, skips empty lanes, and a failed label write warns without failing the job.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Mô tả

needs-triage priority: medium

Problem

On a pull request with more than 250 commits, pr-refine / fix-docs fails even when the lane made no commits.

  • .github/actions/check-signed-commits/check-signed-commits.mjs:14 sets LISTED_COMMITS_CAP=250. Past the cap, line 92 sets complete=false, so allVerified is false even when every listed commit is verified.
  • The escalation then POSTs a label to issues/<n>/labels. That fails with 403 because the token has no issues write, and the failed POST becomes a hard failure.
  • The check runs even when the lane produced no commits ("The activity made no commits"), so there is nothing of the lane's to check.

Observed on #7009 (259 commits), run 38179498135: check-run summary "A commit on the pull request is not GitHub-verified."; step "Check the lane's commits are signed": "POST .../issues/7009/labels failed with status 403".

Expected

  • Skip the signed-commit check when the lane made no commits.
  • Check only the lane's own commits, not the PR's full history, so the 250-commit list cap does not apply.
  • A failed escalation label write should be a warning, not a job failure.
Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
8 giờ 27 phút
Pull request đã merge (30 ngày)
957

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của melodic-software/claude-code-plugins

Tất cả issue của melodic-software/claude-code-plugins

Issue tương tự

Thêm issue về Shell/Bash

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.