secondo voi ciò che ho combinato è corretto? :)
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
lp:openobject-italia/7.0.
___
Mailing list:
+1
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
lp:openobject-italia/7.0.
___
Mailing list: https://launchpad.net/~openobject-italia-core-devs
per il PEP8, il modulo non l'ho scritto io, ma a parte questo direi che sarebbe
inutile al momento siccome è un modulo da rivedere da 0
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
ciò che chiedo di controllare è se il rebase è andato a buon fine, siccome il
precesso mi è un po' oscuro. ho dovuto comunque fare le stesse cosa che avrei
fatto con un semplice merge da openobject-italia/7.0.
quindi presumo di non aver compreso tutto correttamente
--
No...quanto fatto non va bene. ripeto facendo un merge come suggerito da
elbati. ma conviene che riprongo branche e merge
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
A me il diff sembra OK.
Sono però d'accordo nel seguire il PEP8 per le nuovi parti di codice
introdotte. In questo modo non introduciamo altro codice sporco e quando si
farà una ristrutturazione stilistica ci sarà meno lavoro da fare.
--
quando si fa il pep8 sarebbe meglio fare un mp con solo quelle modifiche
altrimenti si obbliga tutti a rileggere tutto il codice quando magari si
è tolto solo uno spazio a fine riga...
On 21/05/2014 16:01, Andrea Cometa wrote:
anche per me il diff va bene
per il PEPOTTO dovremo considerare di
On 05/21/2014 04:12 PM, Nicola Riolini wrote:
quando si fa il pep8 sarebbe meglio fare un mp con solo quelle
modifiche altrimenti si obbliga tutti a rileggere tutto il codice
quando magari si è tolto solo uno spazio a fine riga...
Esatto
--
On 21 May 2014, at 16:12, Nicola Riolini nicola.riol...@gmail.com wrote:
quando si fa il pep8 sarebbe meglio fare un mp con solo quelle modifiche
altrimenti si obbliga tutti a rileggere tutto il codice quando magari si è
tolto solo uno spazio a fine riga…
sono d’accordo
motivo per cui
The proposal to merge lp:~enlightx/openobject-italia/7.0_l10n_it_sale_ext into
lp:openobject-italia/7.0 has been updated.
Status: Needs review = Rejected
For more details, see:
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
--
lapsus, come faccio ad aggiornare questo branch con gli aggiornamenti
provenienti da openobject-italia/7.0 siccome i branch divergono oramai?
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
Nel tuo branch fai
bzr merge lp:openobject-italia/7.0
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
lp:openobject-italia/7.0.
___
Mailing list:
Review: Needs Fixing
riga 625: sarebbe forse meglio mettere noupdate=1? Di modo che se l'utente
modifica o elimina i records questi non vengano sovrascritti all'aggiornamento
(lo stesso discorso varrebbe anche per gli altri campi del DDT)
--
Review: Approve
--
https://code.launchpad.net/~enlightx/openobject-italia/7.0_l10n_it_sale_ext/+merge/203035
Your team OpenERP Italia core devs is subscribed to branch
lp:openobject-italia/7.0.
___
Mailing list:
14 matches
Mail list logo