Phase 1 quick wins: bounded SHA fetches, concurrency-group fallback, paths-ignore in templates/ci.yml

Đã đóng
  • Có gì không ổn?
#147 0 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
76/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Ít trao đổi
Công nghệ
github-actions, shell
Lĩnh vực
ci-cd, devops

Hướng nghiên cứu

Bắt đầu với scripts/detect-changed-lectures.sh:26-27 và templates/ci.yml:24-26, sau đó kiểm tra các workflow trigger của template và hai thành phần sử dụng nó. Thêm các lần fetch SHA có giới hạn, concurrency fallback pull-request-number/run-id và bộ lọc fail-open paths-ignore để các pull request chỉ thay đổi tài liệu bỏ qua previews mà không làm gián đoạn các lần chạy thủ công. Hoàn tất khi cả ba mục trong checklist đều được phản ánh trong template có thể tái sử dụng và script.

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

Mô tả

enhancement

The "one small actions-repo PR" bundle from #92 Phase 1 — three independent quick wins, all in this repo:

  • --depth=1 on the two still-unbounded SHA fetches in scripts/detect-changed-lectures.sh:26-27. Shippable today; lecture-python.myst has no copy of the script (it calls preview-netlify@v0, which invokes ours), so only the checkout side is consumer work — tracked with the consumer items.
  • Concurrency cancellation fallback in templates/ci.yml:24-26: group: ci-${{ github.event.pull_request.number || github.run_id }}. The template has the group and cancel-in-progress: true but not the fallback, and the fallback is load-bearing: both consumers also trigger on workflow_dispatch, so a verbatim copy puts every manual run in the empty group ci-, cancelling each other. Also closes a stale-overwrite hazard.
  • Paths filters (paths-ignore fail-open variant) in templates/ci.yml so docs-only PRs skip the preview — the template has no paths key at all.

Related, deliberately not here: the CLI-install gate on pull_request for both preview actions folds into #105 (same two steps it wants pinned — one PR, not two); the zstd/tag-input item stays separate because :zstd must be validated before :latest moves.

Ngôn ngữ chính
Shell
Star
0
Fork
1
Merge trung bình
18 giờ 51 phút
Pull request đã merge (30 ngày)
58

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 QuantEcon/actions

Tất cả issue của QuantEcon/actions

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.