New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[7.0] [FIX] pep8 tutti i moduli #180
Conversation
👍 Grazie @zeroincombenze Riusciresti a fare lo stesso anche per 8.0, che attualmente è rosso per
|
Volentieri, la mia intenzione in questa prima fase è fare questo lavoro Grazie *zeroincombenze® * SHS-AV Antonio M. Vigliotti SHS-AV s.r.l. *(impresa innovativa) Il 27/01/2016 12:01, Lorenzo Battistini ha scritto:
|
@zeroincombenze grazie, Travis è ancora rosso, vedi Details a fianco della sua build corrispondente, ovvero: https://travis-ci.org/OCA/l10n-italy/jobs/104867048 |
Grazie Alex, zeroincombenze® * SHS-AV Antonio M. Vigliotti SHS-AV s.r.l. *(impresa innovativa) Il 27/01/2016 12:13, Alex Comba ha scritto:
|
Hey @zeroincombenze, thank you for your Pull Request. It looks like some users haven't signed our Contributor License Agreement, yet.
Appreciation of efforts, |
Hello, Wish I send a signed Entity CLA? **zeroincombenze® * SHS-AV s.r.l. *(impresa innovativa) Il 27/01/2016 13:47, OCA CLA BOT ha scritto:
|
I electornically signed Entity CLA Is good for you? zeroincombenze® * SHS-AV Antonio M. Vigliotti SHS-AV s.r.l. *(impresa innovativa) Il 27/01/2016 13:47, OCA CLA BOT ha scritto:
|
Confermo che W8201 è dovuto al flag x dei file.
zeroincombenze® * SHS-AV Antonio M. Vigliotti SHS-AV s.r.l. *(impresa innovativa) Il 27/01/2016 12:13, Alex Comba ha scritto:
|
@zeroincombenze direi, come prima cosa, di puntare a rendere verde la build di Travis, quindi farei:
|
Non ho capito nulla (come al solito :-[ ) |
@zeroincombenze Ciao.
Altrimenti, direi che possiamo anche mergiarla così com'è, correggendo solamente |
Io avevo capito che la regole fosse 1 PR per ogni modulo e quindi Se non avete preferenze, io sistemerei tutta la 7.0 zeroincombenze®*
|
Ok per me. Per la versione 8 di l10n_it_ricevute_bancarie, vedi #185 |
👍 thanks again |
OK, merging! 👍 Thanks for the efforts. |
[7.0] [FIX] pep8 tutti i moduli
(en) Correct pep8 unary operator error.
(it) Corretto error pep8 'unary operator'