SIU-Toba / framework

Framework para desarrollo rápido de aplicaciones web
http://toba.siu.edu.ar
21 stars 24 forks source link

Agrega el procesamiento del parametro de linea de comando --alias-nuc… #153

Closed alfrenovsky closed 1 year ago

alfrenovsky commented 1 year ago

El parametro de linea de comando --alias-nucleo figura en la ayuda de "bin/toba instalacion" pero no era procesado y siempre era preguntado de forma interactiva.

Este commit agrega el procesamiento de ese parametro de linea de comando en la funcion definir_alias_nucleo.

alfrenovsky commented 1 year ago

@alfrenovsky Mas alla del detalle de la indentación, te pido ese cambio asi homogeneizamos el comportamiento con el comando instalacion_silenciosa.

El comando crear (para la instalacion basica) no debe hacer uso del parametro. Gracias.

En ese caso deberia rechazars este pull request. Voy a ver si lo puedo hacer andar con instalacion_silenciosa que tambien me fuerza interacción.

alfrenovsky commented 1 year ago

@alfrenovsky Mas alla del detalle de la indentación, te pido ese cambio asi homogeneizamos el comportamiento con el comando instalacion_silenciosa.

El comando crear (para la instalacion basica) no debe hacer uso del parametro. Gracias.

En instalacion_silenciosa lo correcto sería que tome alias-nucleo del parametro de lines de comando --alias-nucleo o que lo tome del yaml?

enfoqueNativo commented 1 year ago

@alfrenovsky Mas alla del detalle de la indentación, te pido ese cambio asi homogeneizamos el comportamiento con el comando instalacion_silenciosa. El comando crear (para la instalacion basica) no debe hacer uso del parametro. Gracias.

En ese caso deberia rechazars este pull request. Voy a ver si lo puedo hacer andar con instalacion_silenciosa que tambien me fuerza interacción.

Por que motivo?, el bug existe y lo que subiste esta practicamente listo.. solo necesita algun minimo ajuste.

enfoqueNativo commented 1 year ago

@alfrenovsky Mas alla del detalle de la indentación, te pido ese cambio asi homogeneizamos el comportamiento con el comando instalacion_silenciosa. El comando crear (para la instalacion basica) no debe hacer uso del parametro. Gracias.

En instalacion_silenciosa lo correcto sería que tome alias-nucleo del parametro de lines de comando --alias-nucleo o que lo tome del yaml?

Deberia de aceptarlo de la linea de comandos, no recuerdo que se haya definido entrada para el archivo yml.

enfoqueNativo commented 1 year ago

Se cierra por falta de respuesta, se replico el cambio en [3aba795198ae4512eccf8ad6c7dc41ac6cb8becf]