Action CI: cover multi-id only and skip on all three OSes

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#71 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ó
2/5
Thời gian dự kiến
1-3 giờ
Mức phù hợp với người mới
72/100
Loại issue
Tính năng
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
github-actions, markdown, python, yaml
Lĩnh vực
build-system

Hướng nghiên cứu

Bắt đầu từ job trong .github/workflows/ci.yml chạy uses: ./ và phần phân tích đầu vào only/skip trong action.yml. Thêm các bước matrix cho only: project git trên tests/fixtures/action-pass, và cho only: project cộng skip trên tests/fixtures/action-fail, sau đó cập nhật ghi chú “Not covered” trong ROADMAP.md. Hoàn thành khi các bước đều pass trên Ubuntu, Windows và macOS, và một phép tách theo khoảng trắng bị cố ý làm hỏng khiến một bước mới thất bại.

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

Mô tả

good first issue help wanted

Problem: The GitHub Action CI job (.github/workflows/ci.yml, job running uses: ./) only ever passes a single id to only (only: project) and never uses skip. Both inputs are documented as space-separated lists in action.yml, so a quoting or splitting bug on one OS would go unnoticed. ROADMAP.md lists this as not covered.

Do:

  • Add steps to the existing Ubuntu/Windows/macOS matrix job:
    • only: project git on tests/fixtures/action-pass: must succeed.
    • only: project plus skip: <the failing id> on tests/fixtures/action-fail: must succeed, proving skip removes the failure.
    • Optional: format: json + output, then assert with a short python -c that the skipped id is absent and both only prefixes are present.
  • Update the ROADMAP.md "Not covered" note (SARIF is already covered by the existing step, so the note is stale there too).

Done when:

  • The new steps pass on all three OSes.
  • Breaking the space splitting in action.yml makes at least one new step fail.

Notes:

  • No product code changes expected; CI and ROADMAP only.
  • Comment below to claim it and I will assign you.
Ngôn ngữ chính
Python
Star
2
Fork
3
Merge trung bình
10 giờ 51 phút
Pull request đã merge (30 ngày)
40

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 Evalogical-Inc/devReady

Tất cả issue của Evalogical-Inc/devReady

Issue tương tự

Thêm issue về Python

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.