Attestation predicate reads `project_config.project_type`, which `ProjectConfig` does not define
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 62/100
Línea de trabajo
Read packages/darnit-baseline/src/darnit_baseline/attestation/predicate.py alongside the ProjectConfig schema at packages/darnit/src/darnit/config/schema.py, then inspect the predicate callers and existing tests. Add coverage for building a predicate with a real ProjectConfig; done means the predicate no longer raises and its no-config type behavior is decided and documented in the predicate changelog.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
packages/darnit-baseline/src/darnit_baseline/attestation/predicate.py:147 builds the predicate with:
"project_type": project_config.project_type if project_config else "software",
ProjectConfig (packages/darnit/src/darnit/config/schema.py) has a type field, not project_type (create_minimal_config(..., project_type=...) maps its argument to type=). When a project configuration is passed (attestation/__init__.py:22, attestation/generator.py:92), this raises AttributeError instead of producing a predicate. It has gone unnoticed because the common paths pass None and fall back to "software".
Proposed fix:
- Read
project_config.type. - Decide whether the
"software"fallback is right when no configuration exists: it states a project type nobody confirmed. Omitting the field, or recording it as unknown, may be more accurate. This is a predicate field, so the change is observable by attestation consumers and should be noted in the predicate's changelog. - Add a test that builds a predicate with a real
ProjectConfig.
- Lenguaje dominante
- Python
- Estrellas
- 22
- Forks
- 27
- Merge medio
- 1 d 20 h
- PR fusionados (30 d)
- 39
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de darnitdevorg/darnit
-
`darnit validate` crashes on openssf-baseline.toml summarising levelsPosiblemente ocupada @i-am-paradox la tomó hace 2 días. Abiertoneeds triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
darnitdevorg/darnit#589 ·
Los mantenedores suelen responder en 1 día
-
needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
darnitdevorg/darnit#587 ·
Los mantenedores suelen responder en 1 día
-
needs triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
darnitdevorg/darnit#585 ·
Los mantenedores suelen responder en 1 día
-
quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
darnitdevorg/darnit#523 ·
Los mantenedores suelen responder en 1 día
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
darnitdevorg/darnit#516 ·
Los mantenedores suelen responder en 1 día
Todos los issues de darnitdevorg/darnit
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
common-workflow-language/cwl-utils#477 ·
Los mantenedores suelen responder en 1 día
-
Add dependabotAbiertofeat
Dificultad 2/5 Menos de una hora Aptitud para principiantes 72/100
open-nudge/opentemplate#204 ·
Los mantenedores suelen responder en 1 día
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limitPosiblemente ocupada @VascoSch92 la tomó hoy. Abiertopriority:medium ready-for-dev
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
OpenHands/extensions#781 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
curriculum documentation quality
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
githubnext/gh-aw-workshop#4638 ·
Los mantenedores suelen responder en 1 día