riseproject-dev / riseproject-dev/python-wheels
Narrow `pr-trigger.yml` to ignore publish-only changes in workflows
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 0
- Fork
- 0
- Merge trung bình
- 16 giờ 14 phút
- Pull request đã merge (30 ngày)
- 952
Mô tả
After creating #169 , the build-numpy.yml workflow kicked off again despite no new version to build. This is expected due to how the existing checker workflows were designed, and the publish step wasn't going to run anyway since it wasn't on main, but it's wasteful to be running a build in this case. If it's simple enough, we should change the pr-trigger.yml workflow so that it only triggers builds when parts of the script other than the publish job are modified. This may be too complex to justify.
Alternatively, we could do some further rework (once the composite action in #169 is merged) to have it perform the publish steps in a "mock" mode, so that we're testing the publishing steps as much as possible too. It would mean letting the superfluous wheel builds to run though, without further tweaks.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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.
Hướng nghiên cứu
Bắt đầu với workflow pr-trigger.yml và so sánh với build-numpy.yml, sau đó xem lại thay đổi workflow được giới thiệu bởi #169. Cập nhật các điều kiện trigger để những thay đổi chỉ liên quan đến việc publish không khởi chạy build, trong khi các thay đổi script khác vẫn khởi chạy build, và xác minh hành vi workflow sau khi thay đổi.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- github-actions
- Lĩnh vực
- build-system, ci-cd
- Loại issue
- Tính năng
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- 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
- 68/100