Gitwatch: Agregue la opción para incluir una diferencia (truncada) en el mensaje de confirmación

Creado en 28 nov. 2012  ·  7Comentarios  ·  Fuente: gitwatch/gitwatch

Como se implementó en una versión única en ibrahima / @ibrahima en los comentarios sobre esa confirmación, podría ser bueno para algunos casos de uso tener la oportunidad de incluir la diferencia en el mensaje de confirmación.

Dejando de lado cosas como el nombre del archivo y truncándolo a la longitud máxima estándar de 80 caracteres de los mensajes de confirmación "cortos" de git, eso debería ir bien con los cambios que generalmente solo cambian un archivo.

La opción también puede detectar el caso de que se hayan cambiado varios archivos y sin incluir la diferencia real en ese caso, sino algo como "Varios archivos cambiados, diferencia omitida".

help wanted

Comentario más útil

@mlncn Si prueba mi solicitud de extracción https://github.com/gitwatch/gitwatch/pull/54 ( git fetch origin pull/54/head:pr-54 && git checkout pr-54 ), debería poder ver los nombres de archivo (con un gráfico de los cambios) como el confirmar mensaje usando gitwatch -l 1 ~/myproject .

Todos 7 comentarios

podría implementar un cambio de opción (por ejemplo, -d / - diff) para incluir esto, y un cambio adicional para establecer después de cuántos caracteres debería truncarse (debería ser 80 por defecto)

Parece una forma sensata de hacerlo.

La última vez que verifiqué que no había una manera _buena_ de hacer cambios de opciones largos (--diff) en bash (hay algunas formas razonables, ninguna de las cuales considero elegante, pero las secuencias de comandos de bash rara vez son elegantes;)). Dado que esto es bash, probablemente optaría por getopts y solo haría cambios de un solo carácter, aunque hay una forma de hacer opciones largas que se basan en un comportamiento no estándar o algo así. También está GNU getopt que admite opciones largas, pero puede ser una solución menos portátil y parece un poco más fea. De todos modos, probablemente no te importe todo eso en este punto, ya que hasta ahora gitwatch no toma ninguna opción además de la ruta, pero supongo que depende de ti lo que pienses al respecto. He trabajado con getopts antes, es bastante sencillo y está integrado en bash.

Supongo que la otra pregunta es cómo truncar y si desea que se incluyan símbolos + - y si también desea nombres de archivo. Tal vez podría haber algunas opciones que agreguen diferentes componentes al mensaje (diferencia truncada, archivos cambiados, el mensaje actual de confirmación automática, etc.).

Introdujimos algunas opciones cortas en la rama de pruebas, usando getopts en bash. Pero tiene razón, si no hay una forma "buena" de utilizar las opciones largas, las cortas servirán por ahora.

Por ahora, basta con anteponer el mensaje diff antes que el resto del mensaje de confirmación. Los usuarios pueden modificar el script con bastante facilidad si prefieren un tipo diferente de mensaje de confirmación. Si pudiera tirar de la rama de prueba y agregar la opción y el manejo allí, sería increíble: D

Parece que se lograron avances significativos en esto. Si alguien quiere volver a visitar esto y terminarlo, me complacerá considerar fusionarme.

De hecho, estoy buscando algo similar, pero solo quiero incluir el nombre del archivo en el mensaje de confirmación. Probablemente sea más fácil o tal vez ya exista una forma de hacerlo.

@dmusican También quiero esta función, así que la implementé y creé una solicitud de extracción contra su última rama maestra para ella: https://github.com/gitwatch/gitwatch/pull/54
Además, dado que puede establecer la cantidad de líneas que se muestran como "git diff" frente a "git diff --stat", el uso de "-l 1" hará que la funcionalidad que alguien intentó implementar en una solicitud de extracción anterior https: // github.com/gitwatch/gitwatch/pull/23/commits/af28cf4a87c71cdb9da7046d40ea9542a4661f43

@mlncn Si prueba mi solicitud de extracción https://github.com/gitwatch/gitwatch/pull/54 ( git fetch origin pull/54/head:pr-54 && git checkout pr-54 ), debería poder ver los nombres de archivo (con un gráfico de los cambios) como el confirmar mensaje usando gitwatch -l 1 ~/myproject .

Solicitud de extracción combinada. Muchas gracias, @shervinemami .

¿Fue útil esta página
0 / 5 - 0 calificaciones

Temas relacionados

datag picture datag  ·  9Comentarios

miguelarauj1o picture miguelarauj1o  ·  7Comentarios

hongkongkiwi picture hongkongkiwi  ·  3Comentarios

Alwaysin picture Alwaysin  ·  8Comentarios

troyengel picture troyengel  ·  3Comentarios