ci scope: renaming a C file to *.md classifies the PR as docs-only (code=false)
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 86/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 領域
- build-system, ci-cd
調査の方向性
.github/workflows/ci.yml の Scope ジョブから始め、python3 /tmp/sp-critic/repro_scope.py .github/workflows/ci.yml を実行して rename-to-md の結果を再現します。変更されたパスがどのように収集されるかを確認し、docs のみではない古いパスを持つ rename または delete によって code=true が設定されることを検証します。.c-to-.md の fixture を使って Scope のセルフテストを追加または更新し、fix を元に戻したときに失敗することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Found by a blind critic on PR #1310. The same result on main 7706607: this is pre-existing.
The scope job in .github/workflows/ci.yml decides code=true|false from the changed paths. For a rename it sees only the NEW name. git mv src/foo.c notes.md therefore yields code=false, and every code-scoped job (the suite, ASan, the variants) is skipped, although a C source was removed from the build.
Repro: python3 /tmp/sp-critic/repro_scope.py .github/workflows/ci.yml. It builds tiny git histories and runs the real scope step: a code edit gives true, a docs edit gives false, rename-to-md gives false.
Done when:
- a rename or delete whose OLD path is not docs-only yields
code=true(use--name-statusor--no-renames) - a planted fixture (rename
.c→.md) is in the scope step's self-test or CI and goes red with the fix reverted
- 主要言語
- C
- スター
- 3
- フォーク
- 7
- 平均マージ
- 4時間 15分
- マージ済み PR（30日）
- 106
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
InauguralSystems/EigenScript のほかの issue
-
area:lint-tooling bug
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
InauguralSystems/EigenScript#1340 ·
メンテナーはふだん 1 日以内に返信
-
area:stdlib found-by:code-review kind:silent-wrong
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
InauguralSystems/EigenScript#1338 ·
メンテナーはふだん 1 日以内に返信
-
area:lint-tooling found-by:critic kind:docs-drift
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
InauguralSystems/EigenScript#1335 ·
メンテナーはふだん 1 日以内に返信
-
enrolment: decide test_gc_runner_controls.py (exempt vs enrol) and whether floors need a ratchetオープンarea:gates found-by:critic kind:decision
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
InauguralSystems/EigenScript#1280 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
area:docs good first issue kind:docs-drift
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
InauguralSystems/EigenScript#1277 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
InauguralSystems/EigenScript の issue をすべて見る
似ている issue
-
bug needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
netdata/netdata#24062 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
BasedHardware/omi#19463 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
EchoTools/nevr-runtime#30 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
riscv-software-src/riscv-isa-sim#2448 ·
メンテナーはふだん 2 日以内に返信