Redundant commitConvention and repoType added to .all-contributorsrc file on add

Cerrado
  • ¿Algo no está bien?
#369 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@lwasser ya está trabajando en esto.

Desde el 27/10/2026.

  • #542 de @JoshuaKGoldberg — fusionado
  • #546 de @matbrgz — cerrado sin fusionar
  • #585 de @lwasser — abierto

Evaluación

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

Línea de trabajo

Empieza reproduciendo el problema con un .all-contributorsrc mínimo y npx all-contributors-cli add JoshuaKGoldberg doc. Sigue la ruta de actualización de la configuración del comando add y comprueba cómo se escriben los valores predeterminados documentados. Se considera terminado cuando los archivos existentes sin estas propiedades permanecen sin cambios después del comando y la reproducción cuenta con cobertura de pruebas.

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

Descripción

type: bug

Describe the bug

.all-contributorsrc files shouldn't need to include commitConvention or repoType properties. They both have default values as documented on https://allcontributors.org/docs/en/bot/configuration:

  • commitConvention: "angular"
  • repoType: "github"

But, running npx all-contributors-cli add ... is adding commitConvention or repoType properties with their default values to .all-contributorsrc files that don't have them explicitly.

 {
 	"contributors": [ ... ],
 	"projectName": "repros-example",
 	"projectOwner": "JoshuaKGoldberg",
+	"repoType": "github",
+	"commitConvention": "angular"
 }

To Reproduce

See https://github.com/JoshuaKGoldberg/repros/tree/all-contributors-cli-add-field-defaults.

  1. Create a repo with a minimal .all-contributorsrc
  2. Run npx all-contributors-cli add JoshuaKGoldberg doc

Expected behavior

The .all-contributorsrc file shouldn't have commitConvention or repoType added, since they have documented defaults.

Additional context

I filed a bunch of feature requests over on https://github.com/all-contributors/all-contributors for more default-friendly behavior. https://github.com/all-contributors/all-contributors/issues/808 is something of a hub for them.

Lenguaje dominante
JavaScript
Estrellas
425
Forks
152
Merge medio
2 d 20 h
PR fusionados (30 d)
17

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 all-contributors/cli

Todos los issues de all-contributors/cli

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.