commitizen / commitizen/cz-cli
Let the adapters depend on inquirer
- Lenguaje dominante
- JavaScript
- Estrellas
- 17.5k
- Forks
- 566
- Merge medio
- 8 h 16 min
- PR fusionados (30 d)
- 1
Descripción
Instead of us supplying inquirer to the adapters, I think that it would be very nice if they could depend on it themselves. Then they can update that dependency at their own discretion instead of being dependent on the main module.
Currently you can choose to be compatible with either commitizen > 2.8.0, or with versions < 2.8.0 since we upgraded inquirer which broke the api. Since this has already been updated as well in conventional-changelog, I think it's to late to revert the update here.
The problem I'm facing as an adapter author is that if I upgrade, I need to go around and tell everyone at the office to update to a newer version of commitizen. I also must upgrade all of the repos that uses our internal adapter at the same time. Otherwise we will have some repos that works in one version, and some that works in other.
Basically this is quite a lot of pain and I think that we should move towards letting the adapters depend on their dependencies separately.
Also, there might even be a benefit to let our interface to the adapters be completely language agnostic. We could just execute the file, and use the output as the commit message.
@jimthedev I know that you are very busy but if you could just quickly answer if you are in favour of this, then I could implement it 👌
Guía de contribución
Línea de trabajo
No se nombran archivos ni pruebas. Empieza rastreando cómo cz-cli proporciona inquirer a adapters y cómo la salida de adapters se convierte en un mensaje de commit. Define el límite de dependencias de adapters y el comportamiento de ejecución independiente del idioma antes de determinar el alcance de la implementación.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, node.js
- Área
- cli, developer-experience
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100