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

pr-pipeline: clear a feedback-not-landed escalation once the head moves and no lane thread is open

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#10,074 4 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

@melodic-automation-lanes đang làm issue này rồi.

Từ ngày 29/10/2026.

  • #10886 của @melodic-automation-lanes — đang mở

Đánh giá

Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
83/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ệ
javascript
Lĩnh vực
devtools

Hướng nghiên cứu

Đọc .github/scripts/pr-merge/clear-escalations.mjs, đặc biệt là bảng CHECKS, cùng các trường hợp liên quan trong clear-escalations.test.mjs. Thêm bước kiểm tra được yêu cầu bằng cách sử dụng khóa tiếp tục escalation và các luồng review lane chưa được giải quyết, sau đó chạy tệp kiểm thử đó. Công việc hoàn tất khi head đã thay đổi và không có luồng lane nào đang mở thì escalation được xóa, còn head không thay đổi hoặc có luồng lane đang mở thì escalation vẫn được giữ lại.

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

Mô tả

priority: medium status: needs-decision work-class: scoped

Problem

clear-escalations.mjs clears a pr-unblock escalation once its reason no longer holds, but its CHECKS table (.github/scripts/pr-merge/clear-escalations.mjs:120-132) covers only workflow-staged, unresolved-conflict, protected-path-conflict and feedback-protected-path. A feedback-not-landed escalation stays labeled after the cause is gone.

Example: #9965 escalated feedback-not-landed at resume key e5183af3; the head moved to 898665b3 with no unresolved review thread left, and ci-status stayed red only on the needs-human label until a person removed it.

Owner decision (2026-10-25)

A stop is worth a person only for security, intent, or a real charge. feedback-not-landed is none of those once the head has moved.

Fix

Add a feedback-not-landed entry to CHECKS in clear-escalations.mjs: the reason no longer holds when the head differs from the escalation's resume key and no review thread the lane replied to is left unresolved. Reuse the existing clearing path; add no second mechanism.

Done when

  • clear-escalations.test.mjs covers: head moved and no open lane thread clears; head unchanged keeps; an open lane thread keeps.
Ngôn ngữ chính
Shell
Star
22
Fork
2
Merge trung bình
3 giờ 38 phút
Pull request đã merge (30 ngày)
959

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.