ci: Pages deploy can be cancelled mid-run by cancel-in-progress on main
Maintainer thường phản hồi trong vòng 1 ngày
Một pull request liên quan đã được merge.
- #147 của @ajithakdev — đã merge
Đá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
- 84/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, yaml
- Lĩnh vực
- ci-cd
Hướng nghiên cứu
Bắt đầu với .github/workflows/ci-cd.yml và kiểm tra các thiết lập đồng thời của workflow cùng với job deploy chạy actions/deploy-pages. Giữ nguyên hành vi đã nêu đối với pull request và đảm bảo rằng lần push thứ hai lên main được xếp hàng trong khi một lần triển khai Pages đang chạy. Hoàn thành khi cả hai tiêu chí chấp nhận đều được đáp ứng.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
.github/workflows/ci-cd.yml sets concurrency: { group: ${{ github.workflow }}-${{ github.ref }}, cancel-in-progress: true } for the whole workflow, including the deploy job that runs actions/deploy-pages. Two quick pushes to main cancel the first run, even if it is already in the deploy job. A cancelled Pages deployment can leave the deploy in a failed state and shows a red run on main. GitHub's own Pages starter workflow uses cancel-in-progress: false for this reason.
Proposed fix
Keep cancelling superseded PR runs, but not deploys on main:
concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
cancel-in-progress: ${{ github.event_name == 'pull_request' }}
Or move the deploy job into its own pages concurrency group with cancel-in-progress: false.
Acceptance criteria
- A second push to a PR still cancels the first PR run.
- A second push to
mainqueues behind a running deploy instead of cancelling it.
- Ngôn ngữ chính
- TypeScript
- Star
- 2
- Fork
- 1
- Merge trung bình
- 1 ngày 4 giờ
- Pull request đã merge (30 ngày)
- 32
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- 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 ajithakdev/toolglass
-
devex
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
ajithakdev/toolglass#145 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
documentation
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
ajithakdev/toolglass#143 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
javascript testing
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 72/100
ajithakdev/toolglass#138 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
enhancement later new tool
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
ajithakdev/toolglass#128 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
feat: Toolglass Chrome/Edge/Firefox Extension — instant omnibox launcher and context-menu utilitiesĐang mởenhancement later new tool ux
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
ajithakdev/toolglass#105 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của ajithakdev/toolglass
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
awesome-dsh-plugin/awesome-dsh-plugin#9184 ·
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 75/100
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 85/100
farion1231/cc-switch#8829 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[core-client] Remove unused hand-maintained transport helpers after the definition-table cutoverĐang mởdoc-audit enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 88/100
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 85/100