Rename the ten phase/faz-named test modules to feature names and add a guard against `test_(phase|faz)N` file names
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 55/100
調査の方向性
The ten phase-named modules are under tests/ (for example tests/test_phase7.py and tests/test_faz27_narrow_exceptions.py); grep for each name to find its code comments, standards, review-pr skill mirrors and ci.yml references. tools/check_test_naming.py is new and must reject names matching test_(phase|faz)\d+, then be wired into CI and the gauntlet. Done when the renames use git mv, no live reference uses the old names, the guard passes in CI, and the row is removed from docs/roadmap/risks-and-decisions.md with the ID named in the CHANGELOG.
索引モデルが issue の本文から書いたものです。
説明
Deferred work ·
F-PR29-A4-04· severity Low · cost ~2 h · recorded indocs/roadmap/risks-and-decisions.md:82
Summary
testing.md asks for one test_<module>.py per forgelm/<module>.py where practical, but ten test modules are named after review phases — test_phase7.py, test_phase12_5.py, test_phase12_review_fixes.py, test_data_audit_phase12.py, test_ingestion_phase12.py, test_cli_phase10.py, test_wizard_phase11.py, test_wizard_phase11_5.py, test_faz27_narrow_exceptions.py and, added after the deferral, test_wizard_phase22.py. With no guard the pattern keeps spreading.
Condition or budget
None recorded (the row still names v0.6.x).
State today (re-checked 2026-10-09)
All ten files are under tests/; tools/check_test_naming.py does not exist.
Acceptance criteria
- Each module is renamed with
git mv(history preserved) or folded into the matchingtest_<module>.py, and every live reference (code comments, standards, both review-pr skill mirrors,ci.yml) moves with it. -
tools/check_test_naming.pyrejectstest_(phase|faz)\d+names and is wired into CI and the gauntlet. - The row is removed from the deferral table and the CHANGELOG names its ID (the table's removal contract).
Recorded in docs/roadmap/risks-and-decisions.md (read at f94595f) and re-checked against the code on 2026-10-09. The row leaves that table only when the fix is on main, the CHANGELOG names its ID and any promised guard exists.
- 主要言語
- Python
- スター
- 9
- フォーク
- 1
- 平均マージ
- 8時間 10分
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
HodeTech/ForgeLM のほかの issue
-
area: dev-tooling bug severity: low source: roadmap wave: 4
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
area: site bug good first issue severity: low source: review-2026-09 wave: 4
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
-
documentation good first issue severity: medium source: review-2026-09 wave: 3
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
documentation good first issue severity: low source: review-2026-09 wave: 4
難易度 1/5 1〜3時間 初心者へのやさしさ 80/100
-
documentation severity: medium source: review-2026-09 wave: 3
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
HodeTech/ForgeLM の issue をすべて見る
似ている issue
-
難易度 1/5 1〜3時間 初心者へのやさしさ 88/100
gradio-app/gradio#13964 ·
メンテナーはふだん 1 日以内に返信
-
難易度 1/5 1時間未満 初心者へのやさしさ 93/100
BSData/horus-heresy-3rd-edition#3232 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
jbacalso24/mimry#123 ·
メンテナーはふだん 1 日以内に返信
-
p:3-mid pydanty:bug pydanty:is-working
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
pydantic/pydantic-ai#10153 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
good first issue priority: medium size: S testing
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
cubrid-lab/cubrid-cookbook-python#316 ·
メンテナーはふだん 1 日以内に返信