MozillaItalia / firefox-vademecum

A little italian 2 pages manual for users that want to discover Firefox and the community
6 stars 8 forks source link

Script reso invocabile da qualsiasi percorso #139

Closed Gelma closed 4 years ago

Gelma commented 4 years ago

Prima:

✔ /tmp/fv [fix_invocazione L|✔] 12:51 $ bash scripts/genera_pdf_VG_VT_CV.sh || Starting conversion for versione_generale (VG) || Pdf files generating... QPainter::begin(): Returned false QPainter::begin(): Returned false Pdf files generated. Png files generating... Loading page (1/2) Rendering (2/2) Error: Could not write to output file Error: Could not save image Done Exit with code 1, due to unknown error.

Dopo:

✔ /tmp/fv [fix_invocazione L|✚ 1] 12:55 $ /tmp/fv/scripts/genera_pdf_VG_VT_CV.sh || Starting conversion for versione_generale (VG) || Pdf files generating... Pdf files generated. Png files generating... Loading page (1/2) Rendering (2/2) Done Loading page (1/2) Rendering (2/2) Done Png files generated. Pdf files merging in a single pdf file... Pdf merged correctly. Temp files deleting... Temp files deleted. || Completed conversion for version VG || || Starting conversion for versione_tecnica (VT) || Pdf files generating... Pdf files generated. Png files generating... Loading page (1/2) Rendering (2/2) Done Loading page (1/2) Rendering (2/2) Done Png files generated. Pdf files merging in a single pdf file... Pdf merged correctly. Temp files deleting... Temp files deleted. || Completed conversion for version VT || || Completed all conversions ||

Details: NAME: Vademecum VERSION: 2.0 VERSION TYPES GENERATED: | VG: | | Vademecum_2.0_VG_fronte.png | | Vademecum_2.0_VG_retro.png | | Vademecum_2.0_VG.pdf | VT: | | Vademecum_2.0_VT_fronte.png | | Vademecum_2.0_VT_retro.png | | Vademecum_2.0_VT.pdf

Sav22999 commented 4 years ago

Come nella precedente PR: invece di fare PR dirette, aprire prima un issue. Inoltre, descrivere la procedura per ottenere l'errore

Gelma commented 4 years ago

Come nella precedente PR: invece di fare PR dirette, aprire prima un issue. Inoltre, descrivere la procedura per ottenere l'errore

Non capisco perché passare per forza dalla issue. Ma è un limite mio. Io sono stupido.

Tranquillamente, buttate pure il tutto. No drama.

Gelma commented 4 years ago

Come nella precedente PR: invece di fare PR dirette, aprire prima un issue. Inoltre, descrivere la procedura per ottenere l'errore

La procedura per ottenere l'errore è completamente copia/incollata nel messaggio di commit.

Sav22999 commented 4 years ago

Con gli issue possiamo tracciare più facilmente il lavoro che facciamo. Non è che siano inutili le tue PR, ho sbagliato a esprimermi. Volevo dire che, invece di fare molte PR, è meglio aprire più issue e, successivamente, fare una PR che ingloba più issue 😄. Ad esempio, avresti potuto fare una unica PR, sia per correggere readme, sia per correggere questo bug nel codice dello script. Ecco tutto. Grazie mille per il contributo, in ogni caso