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

オープン 初心者向け
  • 何か問題がありますか？
#71 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
72/100
issue の種類
機能追加
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
github-actions, markdown, python, yaml
領域
build-system

調査の方向性

.github/workflows/ci.yml の uses: ./ を実行するジョブと、action.yml の only/skip 入力の解析から始めてください。tests/fixtures/action-pass に対して only: project git、tests/fixtures/action-fail に対して only: project と skip を指定するマトリクスステップを追加し、その後 ROADMAP.md の「Not covered」の注記を更新してください。ステップが Ubuntu、Windows、macOS で成功し、意図的に壊した空白による分割で新しいステップが失敗すれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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.
主要言語
Python
スター
2
フォーク
3
平均マージ
10時間 51分
マージ済み PR（30日）
40

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

Evalogical-Inc/devReady のほかの issue

Evalogical-Inc/devReady の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。