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

Chiusa Adatta ai principianti
#709 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 3 giorni

Una pull request collegata è già stata integrata.

  • #713 di @JohnsonLyu — integrata

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
68/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
cli

Direzione di ricerca

Il rifiuto si trova in cmd_plan, e il suo commento cita #568, che è chiuso. Confrontalo con cmd_audit, che già restringe i controlli tramite _resolve_profile_ids. Riproduci con darnit plan --profile level1_quick <repo> e conferma che termina con codice 1. È completato quando plan accetta il profilo, stampa solo i controlli selezionati e restituisce gli stessi errori di audit per un profilo sconosciuto o una selezione vuota.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

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.

Lingua principale
Python
Stelle
22
Fork
27
Merge medio
4g 20h
PR unite (30g)
75

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di darnitdevorg/darnit

Tutte le issue di darnitdevorg/darnit

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.