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

Merge queue squash commit fails required_signatures when a wrapped PR body line starts with 'committer '

Đang mở
  • Có gì không ổn?
#7,279 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ó
3/5
Thời gian dự kiến
1-2 ngày
Mức phù hợp với người mới
55/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
git, github, github-actions, shell
Lĩnh vực
ci-cd, devtools

Hướng nghiên cứu

Start by inspecting the /source-control:pull-request body tooling and the pr-contract CI check mentioned in the issue. Reproduce the wrapping at 72 columns and check whether any wrapped body line starts with a Git commit header keyword. Done means the queue no longer creates an invalid signed squash commit for this case, with a check that catches the offending line before merge.

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

Mô tả

needs-human

The merge queue removed #7225 twice with branch_protection_failure, even though its CI passed in the queue.

What happened

  • The queue's squash commit c2b54b0, committed by GitHub, shows verification.reason: invalid. The required_signatures rule (org ruleset signing, 17989003) then refuses it.
  • GitHub builds the squash message from the PR body and wraps it at about 72 columns. One wrapped line began committer identity set in the step env.
  • In the signed payload, the gpgsig block sits right after that body line, not after the real committer header. So the signing placed the signature after the first line starting committer .
  • The previous squash commit on main, 4a453b0, is valid.
  • Rewording the body so no wrapped line starts with committer fixed it.

Who acts and what they do

PR authors and the PR body tooling (/source-control:pull-request) should keep any line of the wrapped body from starting with a git header keyword, committer above all. One option is a ci-status pr-contract check that wraps the body at 72 columns and fails on ^(tree|parent|author|committer|gpgsig|encoding|mergetag) . That check would catch it before the queue does.

Evidence: gh api repos/melodic-software/claude-code-plugins/git/commits/c2b54b03e9c0bcacbf7367a2fa923d3ee308bcf9 --jq .verification

🤖 Generated with Claude Code

Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
8 giờ 34 phút
Pull request đã merge (30 ngày)
946

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.