pr-pipeline: apply-patch held() honours a Hold: line added to an adopted owner PR mid-run
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ó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 73/100
Hướng nghiên cứu
Bắt đầu với apply-patch.mjs và đọc held() quanh các dòng 180–186, sau đó kiểm tra cách xử lý PR do chủ sở hữu tạo được mô tả trong #8997. Thêm một bài kiểm tra trong đó PR do chủ sở hữu tạo nhận được một dòng Hold: giữa dispatch và push; hoàn thành khi lượt chạy đó không push PR ấy và hành vi hold hiện có đối với PR của Lane App vẫn được giữ nguyên.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Refs: #8997
apply-patch.mjs held() (~:180-186) re-checks a PR's draft state and hold labels at push time, but not a body line starting Hold:. Since #8997, fix-ci adopts idle owner PRs and skips one that carries a Hold: line at dispatch; a Hold: line added while a fix-ci run is already going does not stop that run's push. The next dispatch respects it.
Fix: have held() read the PR body and treat a line starting Hold: as held, for adopted owner PRs (lanes App PRs carry Hold: a human merges this, which holds the merge, not a CI fix, so scope the check to owner-authored PRs as #8997 does). Test that fails first: an owner PR whose body gains a Hold: line between dispatch and push is not pushed.
From the S&C review of #8997 (security OK at 69983e27; this was the accepted residual).
- Ngôn ngữ chính
- Shell
- Star
- 22
- Fork
- 2
- Merge trung bình
- 7 giờ 23 phút
- Pull request đã merge (30 ngày)
- 950
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- 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.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của melodic-software/claude-code-plugins
-
fix(session-flow): redact.py misses bare sk-ant-oat01- tokens and stateless ghs_ tokens with dotsĐang mởneeds-human
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
melodic-software/claude-code-plugins#9026 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-human
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
melodic-software/claude-code-plugins#9013 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
needs-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
melodic-software/claude-code-plugins#9000 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
fix(source-control): ready hold check treats hyphenated identifiers containing wip as a WIP markerĐang mởpriority: medium
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 77/100
melodic-software/claude-code-plugins#8977 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
fix(ai-slop): detect.sh --paths-file drops the last entry when the file has no trailing newlineĐang mởpriority: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 86/100
melodic-software/claude-code-plugins#8974 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của melodic-software/claude-code-plugins
Issue tương tự
-
enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
area:terraform priority:low source:reader type:enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
stephrobert/blog-roadmap#104 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
ReproNim/containers#172 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
fairagro/m4.2_infrastructure#54 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày