Skip to content
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

feat: build automatizzata #147

Closed
wants to merge 12 commits into from
Closed

feat: build automatizzata #147

wants to merge 12 commits into from

Conversation

dag7dev
Copy link
Member

@dag7dev dag7dev commented Aug 31, 2021

Close #115

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

.github/workflows/vademecum.yaml Outdated Show resolved Hide resolved
@Mte90
Copy link
Member

Mte90 commented Aug 31, 2021

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
Copy link
Member Author

dag7dev commented Aug 31, 2021 via email

@Mte90
Copy link
Member

Mte90 commented Aug 31, 2021

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
Copy link
Member

Mte90 commented Sep 6, 2021

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
Copy link
Member Author

dag7dev commented Sep 6, 2021

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

  • cartella finale
  • cartella "con i file buildati"
    Se non ci sono diff, ad esempio, gli facciamo ritornare 0, altrimenti 1.

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

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

@Mte90
Copy link
Member

Mte90 commented Sep 6, 2021

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
Copy link
Member Author

dag7dev commented Sep 6, 2021 via email

@Mte90
Copy link
Member

Mte90 commented Sep 6, 2021

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
Copy link
Member Author

dag7dev commented Sep 6, 2021 via email

@Mte90
Copy link
Member

Mte90 commented Sep 7, 2021

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

@dag7dev dag7dev closed this Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Usare GH Action per generare pdf
3 participants