nodejs / nodejs/node-core-utils

Allow `request-ci` on the collaborator's PR without restriction of at least 1 approval

Đang mở
#937 6 bình luận 0 reaction 0 người được giao Xem trên GitHub

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

Ngôn ngữ chính
JavaScript
Star
314
Fork
137
Merge trung bình
2 ngày 8 giờ
Pull request đã merge (30 ngày)
24

Mô tả

IIRC collaborators are able to manually trigger CI through Jenkins without needing 1 approval on the PR, maybe it makes sense for the collaborators to use request-ci label on their own or other collaborators's PR without at least 1 approval restriction.

https://github.com/nodejs/node-core-utils/blob/a68af4b7744e8416bfc0f062570688b67d5e10dc/lib/pr_checker.js#L560 just an idea - could we check either the PR author is a current collaborator?

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

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.

Hướng nghiên cứu

Bắt đầu trong lib/pr_checker.js, khoảng dòng 560, và lần theo hạn chế phê duyệt được label request-ci sử dụng. Kiểm tra cách trạng thái collaborator và quyền tác giả của pull request được biểu diễn, sau đó xác minh các kiểm tra nhãn CI liên quan. Hoàn thành nghĩa là một collaborator đủ điều kiện có thể sử dụng request-ci trên pull request của chính họ hoặc của một collaborator khác mà không cần một phê duyệt.

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

Đánh giá

Công nghệ
javascript
Lĩnh vực
authorization, ci-cd
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
56/100

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.