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

Cerrado Apto para principiantes
#709 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 3 días

Ya se ha fusionado un pull request relacionado.

  • #713 de @JohnsonLyu — fusionado

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
cli

Línea de trabajo

La negativa está en cmd_plan, y su comentario cita #568, que está cerrado. Compáralo con cmd_audit, que ya restringe los controles mediante _resolve_profile_ids. Reproduce el problema con darnit plan --profile level1_quick <repo> y confirma que termina con código 1. Se considera terminado cuando plan acepta el perfil, imprime solo los controles seleccionados y devuelve los mismos errores que audit para un perfil desconocido o una selección vacía.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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.

Lenguaje dominante
Python
Estrellas
22
Forks
27
Merge medio
4 d 20 h
PR fusionados (30 d)
75

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de darnitdevorg/darnit

Todos los issues de darnitdevorg/darnit

Issues similares

Más issues de Python

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.