Installers: shellcheck fails on the BReg, Casework, and evidencectl installers
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 86/100
調査の方向性
crates/registry-breg/install.sh、crates/registry-casework/install.sh、crates/registry-evidencectl/install.sh にあるフラグ付きの文を、crates/registry-scheduling/install.sh にある修正済みの形式と比較します。5つの crates/*/install.sh ファイルと editors/install.sh に対して shellcheck を実行し、続けて release/scripts/test_macos_installers.py、release/scripts/test_nightly_release.py、および crate の install_script テストを実行します。すべてのチェックに合格すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
What
shellcheck exits non-zero on three of the four product installers on main (checked 2026-10-03), with the same two warnings in each:
| File | SC2034 | SC2155 |
|---|---|---|
crates/registry-breg/install.sh |
line 329 | line 500 |
crates/registry-casework/install.sh |
line 327 | line 498 |
crates/registry-evidencectl/install.sh |
line 333 | line 515 |
- SC2034:
IFS=. read -r release_major release_minor release_patch <<<"${version#v}"readsrelease_patchand never uses it. - SC2155:
local archive="$tmpdir/$(asset_name "$binary")"declares and assigns in one statement, so a failingasset_nameis masked bylocal's own exit status andset -edoes not stop the script.
crates/registry-scheduling/install.sh and editors/install.sh pass. The Scheduling installer has the same two statements in the corrected form, from #1854:
IFS=. read -r release_major release_minor _ <<<"${version#v}"
local archive
archive="$tmpdir/$(asset_name "$binary")"
Both lines are hand-written; neither sits inside the blocks generated by release/scripts/render-installer-channel.py or release/scripts/render-installer-libc-preflight.py.
Why it is filed rather than fixed
It was found while changing the Scheduling installer in #1854, and the other three installers were outside that change.
No workflow under .github/workflows runs shellcheck, which is how the installers drifted apart. Whether to add a gate is a separate decision from the two-line fix, so it is listed as optional below.
Done when
shellcheckpasses on all fivecrates/*/install.shfiles andeditors/install.sh.- The installer test suites still pass (
release/scripts/test_macos_installers.py,release/scripts/test_nightly_release.py, and each crate'sinstall_scripttests). - Optional: a CI step runs
shellcheckon the installers so the next copy does not drift.
- 主要言語
- Rust
- スター
- 2
- フォーク
- 0
- 平均マージ
- 9時間 14分
- マージ済み PR（30日）
- 241
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
registrystack/registry-stack のほかの issue
-
agent-ready area:breg bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
registrystack/registry-stack#1941 ·
メンテナーはふだん 1 日以内に返信
-
area:casework bug criticality:p2 rust
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
registrystack/registry-stack#1936 ·
メンテナーはふだん 1 日以内に返信
-
area:casework bug criticality:p3 rust
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
registrystack/registry-stack#1934 ·
メンテナーはふだん 1 日以内に返信
-
area:release area:scheduling bug criticality:p3 triage:needs-implementation
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
registrystack/registry-stack#1909 ·
メンテナーはふだん 1 日以内に返信
-
area:breg bug criticality:p3
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
registrystack/registry-stack#1851 ·
メンテナーはふだん 1 日以内に返信
registrystack/registry-stack の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 5 日以内に返信
-
✨ enhancement needs-discussion
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
virtio-fs (Linux passthrough): debug log in do_lookup panics the fs worker on non-UTF-8 file names対応中かも @zcl-g5 が今日担当しました。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
メンテナーはふだん 2 日以内に返信
-
area:docs documentation good first issue priority:low
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信