2014/1/12 oscar_alvarez <[email protected]>: > Hola Devs > > Desde hace tiempo vengo haciendo aportes a tryton y todos los modulos a > traves de la traduccion es_CO y no he tenido problemas, pero con la > exportacion de parches al trunk para hacer contribuciones (no traducciones) > he tenido algunos tropiezos, aunque he tratado de hacerlo siguiendo la guia > en la wiki [1] al pie de la letra he obtenido este mensaje de Cedk: > > This is not what we call submitting a patch, see > https://code.google.com/p/tryton/wiki/HowtoContribute > > > eso fue respecto a un patch que intente enviar hace ya unos meses con nombre > de archivo: issue_864002.patch (usando el commando export de mercurial), el > archivo quedo así: > > # HG changeset patch > # User Oscar Alvarez <[email protected]> > # Date 1368324877 18000 > # Node ID 21c0fc7f22792d89e7b7589136f99a7bd4c756b1 > # Parent d3c06443a7b696aff77ec38c3e3fcbcf87ea5192 > Set correct invoice_type on invoice line > > diff -r d3c06443a7b6 -r 21c0fc7f2279 sale.py > --- a/sale.py Mon Apr 22 12:14:19 2013 +0200 > +++ b/sale.py Sat May 11 21:14:37 2013 -0500 > @@ -1249,6 +1249,7 @@ > invoice_line.product = self.product > invoice_line.unit_price = self.unit_price > invoice_line.taxes = self.taxes > + invoice_line.invoice_type = invoice_type > if self.product: > invoice_line.account = self.product.acco > unt_revenue_used > if not invoice_line.account: > exporting patch: > <fdopen> > > > Aunque fue aceptado el codereview [2] con un LGTM, fue rechazado el archivo > del patch por la forma en que lo envie o por como quedo el archivo, no > entiendo realmente cual fue la causa del rechazo si aparentemente segui la > guia de la wiki y esta guia: > http://mercurial.selenic.com/wiki/SpanishTutorialExport > > [1] > http://code.google.com/p/tryton/wiki/HowtoContribute#How_to_submit_your_patches/contributions > [2] http://codereview.tryton.org/864002/ > > Me gustaria saber ya que algunos de ustedes tienen mas experiencia, cual fue > el error y como deberia entonces ser enviado, debido a este inconveniente > estoy limitado en los aportes que puedo hacer, recuerden que yo soy novato > en el tema de revisión de código.
No añadistes el codereview en el mensaje del commit y si había incidencia bugs.tryton.org tampoco estaba. -- Albert Cervera i Areny Tel. 93 553 18 03 @albertnan www.NaN-tic.com
