search

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

feat: build automatizzata #147

Closed dag7dev closed 5 months ago

dag7dev commented 2 years ago

Close #115

Finalmente quando faremo il push di qualcosa verranno generati PDF e PNG automaticamente ! :D

Mte90 commented 2 years ago

Il mio unico dubbio è fare che la CI committi i file modificati solo se effettivamente sono diversi ai già esistenti. In questo modo non ci troviamo un commit automatico che rigenera i pdf per ogni cosa ogni volta.

dag7dev commented 2 years ago

In che senso?

A parte che possiamo triggerarlo manualmente, ma diciamo che l'idea era che "ogni volta che modifichiamo i file vengono buildati automaticamente". Cioè l'actions l'ho fatta proprio per fare questa cosa.

Non ho ben capito che intendi...

Mte90 commented 2 years ago

Cioè se vedi tu hai modificato il file dell'action ed è stata eseguita anche se non serviva e ha committato dei file che non servivano perchè non sono stati modificati. Dovremmo poter dire all'action di verificare se il commit include dei file in una cartella per poter essere eseguita, così è tutto più pulito e non abbiamo uno storico di commit che non comporta modifiche di nessun tipo.

Mte90 commented 2 years ago

Mi sono ricordato che ho un action che verifica se i file sono diversi prima di fare il commit.

https://github.com/Mte90/Mte90/blob/master/.github/workflows/build.yml#L35

Qui puoi trovare il codice così chiudiamo questo task

Vedo anche altri che hanno risolto così https://github.community/t/commit-push-only-if-files-updated-during-action/156084

dag7dev commented 2 years ago

@Mte90 non possiamo farlo con le actions, dovremmo modificare lo script e buildare in un'altra cartella così poi possiamo fare diff tra:

Nella actions vediamo il risultato dello script e pushiamo di conseguenza.

EDIT: questo perché i file buildati "sovrascrivono" quelli già presenti

Mte90 commented 2 years ago

Casomai facciamo che lo script build.sh accetta un parametro che è il percorso, e nella action usiamo la cartella /tmp come parametro e poi facciamo la comparazione dei file.

dag7dev commented 2 years ago

Anche. Insomma quello che volevo dire è che va messo nello script in ogni caso. Non so se GH Actions pusha comunque o no...

Mte90 commented 2 years ago

Nessun problema modifichiamolo e poi nel file di workflow facciamo un confronto che fa il push senza la action. Non so se si può bloccare il workflow e così non eseguiamo l'action (così sono meno modifiche)

dag7dev commented 2 years ago

A questo punto la action diventa totalmente inutile

Mte90 commented 2 years ago

Vedi tu quale soluzioni prefefrisci, per me serve di non committare i nuovi file se non è necessario.