Prepare release 3.2.7 #60
Conversation
a1b4
requested review from
leosoto,
afiebig,
gdespirito,
eugenioLopezRamos,
saengate and
iescaida
| @@ -5,7 +5,7 @@ | |||
| "transbank/transbank-sdk": ">=1.5.3" | |||
| }, | |||
| "type": "magento2-module", | |||
| "version": "3.2.6", | |||
| "version": "3.2.7", | |||
There was a problem hiding this comment.
En general, no es buena practica utilizar el campo "version" de composer si la libreria se distribuye por Github.
Magento lo hace en sus desarrollos internos porque mantiene un repositorio no composerizado en github y modulos en composer en un repositorio interno (igualmente es bastante fea la solucion).
Lo ideal, que genera menos conflicto y posibilidades de error, es solamente utilizar tags de Git para controlar las versiones y eliminar el campo version del composer.json
| @@ -1,6 +1,6 @@ | |||
| <?xml version="1.0"?> | |||
| <config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:Module/etc/module.xsd"> | |||
| <module name="Transbank_Webpay" setup_version="3.2.6"> | |||
| <module name="Transbank_Webpay" setup_version="3.2.7"> | |||
There was a problem hiding this comment.
En general, no conviene actualizar la version de los instaladores a la par de la version del modulo ya que durante el proceso de deploy, se valida la vigencia de la version existente de schema y datos para cada modulo.
En caso de detectar diferencias, se requiere realizar un setup:upgrade, que involucra downtime de la tienda.
Es decir, el control de versiones del modulo, indica mejoras al modulo.
El control de versiones de setup, indica a Magento que debe ejecutar instaladores al momento de actualizar el modulo.
Lo ideal es sólo modificar el numero de version de setup, en las actualizaciones que efectivamente involucren modificaciones en la base de datos para evitar downtime inecesarios al realizar un deploy.
you can see full changes here 3.2.6...chore/prepare-release-3.2.7