Code review comment for lp:~pexego/openerp-spain/removes_out_of_scope_modules_61

Pedro Manuel Baeza (pedro.baeza) wrote :

Estoy totalmente de acuerdo con Omar. De hecho, no había propuesto algo similar desde que Avanzosc los añadió en la rama por falta de tiempo. Creo que ganaríamos tener que realizar menos esfuerzo cada vez que se deba adaptar esos módulos a una nueva versión si la comunidad entera está implicada, aparte de evitar duplicidades con posibles versiones desactualizadas como ya ha ocurrido en el pasado (recordad aquella pregunta de la lista sobre qué rama utilizar para la localización).

Yo podría echar una mano en los merge-proposals a las ramas correspondientes genéricas, aunque sería necesario renombrar alguno de los módulos como el nan_account_bank_statement para cumplir con las reglas establecidas en la comunidad.

review: Approve

« Back to merge proposal