darnit plan --profile still refuses after #597 fixed the reason

クローズ 初心者向け
#709 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

関連するプルリクエストがすでにマージされています。

  • #713 @JohnsonLyu による — マージ済み

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
68/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
cli

調査の方向性

拒否処理は cmd_plan にあり、そのコメントで参照されている #568 はクローズ済みです。すでに _resolve_profile_ids でコントロールを絞り込んでいる cmd_audit と比較してください。darnit plan --profile level1_quick <repo> で再現し、終了コードが 1 になることを確認してください。plan がプロファイルを受け付け、選択されたコントロールのみを出力し、未知のプロファイルや空の選択に対して audit と同じエラーを返せば完了です。

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

説明

module-core needs triage

darnit plan --profile still refuses, and the reason it gives no longer applies.

#576 made darnit audit --profile filter the controls and left plan refusing the flag, because plan crashed on frameworks whose controls carry no level (#568), so the flag couldn't be wired in a way that could be tested on the Baseline. #597 fixed that crash and closed #568.

On main (155dcc1b), on jonschlinkert/is-odd:

  • darnit plan <repo> exits 0 and prints the plan.
  • darnit plan --profile level1_quick <repo> exits 1 with darnit plan does not support --profile yet (#568); use darnit audit --profile, which points at a closed issue.
  • darnit audit --profile level1_quick <repo> audits 26 of the 66 controls.

Suggested fix: narrow the plan with _resolve_profile_ids as cmd_audit does, with the same errors for an unknown profile or an empty selection, and remove the refusal and its comment in cmd_plan.


Drafted with AI assistance (Claude), reviewed and filed by me. Measurements quoted were executed and verified.

主要言語
Python
スター
22
フォーク
27
平均マージ
4日 20時間
マージ済み PR(30日)
75

環境構築

はじめの一歩

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

darnitdevorg/darnit のほかの issue

darnitdevorg/darnit の issue をすべて見る

似ている issue

Python の issue をもっと見る

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

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