ARGOMENTO: Call for review

Ciao,

invito tutto gli interessati ai moduli di localizzazione ad aiutarci con le review delle PR.

Ecco quelle etichettate come 'needs review':
github.com/OCA/l10n-italy/pulls?q=is%3Ap...3A%22needs+review%22

Maggiori info sul processo di review:
odoo-community.org/page/review

Ciao eLBati,
se vuoi invece di eseguire il pep8 sulle version 8.0 e 9.0 potrei prendermi in carico una PR semplice, in modo da diventare sempre più utile alla community.

Dammi un consiglio

Antoniov ha scritto:
Ciao Antonio,

potresti guardare fra quelle PR che indicavo, prendere una che ti piace e testarla o anche solo guardare il diff e dire se approvi la modifica.

Prendo un esempio semplice:
github.com/OCA/l10n-italy/pull/201

Qui nella descrizione ci sono i passi per riprodurre il problema. Ciò vuol dire che se si prende il modulo l10n_it_ricevute_bancarie attualmente pubblicato su 8.0 e si provano ad eseguire quei passi, si ottiene l'errore.

Nel diff
github.com/OCA/l10n-italy/pull/201/files
cè la modifica proposta

Le review possono essere essenzialmente di 2 tipi: visivo o test

Nella review visiva uno guarda il diff e sulla PR scrive
:+1:

Altrimenti si può testare il modulo usando runbot.
In fondo alla PR c'è il link relativo a 'ci/runbot', poi si clicca su

e si può entrare in un'istanza odoo dove è possibile testare la modifca.

Anche in questo caso, se il test è soddisfacente, sulla PR si scrive
:+1:

Per maggiori info: odoo-community.org/page/review

Grazie Lorenzo,
mi sembra un buon test per iniziare.

vedo che le istanze più vecchie di runbot vengono spente
non mi risulta chiaro come riavvarle per testare le PR

ad esempio volevo testare questa: github.com/OCA/l10n-italy/pull/182

ok, force rebuild
ci ha messo un'oretta ma era quello