Skip to content

Nuove azioni GitHub#46

Merged
virtualdj merged 3 commits into
masterfrom
feat-github-actions
May 21, 2024
Merged

Nuove azioni GitHub#46
virtualdj merged 3 commits into
masterfrom
feat-github-actions

Conversation

@virtualdj

Copy link
Copy Markdown
Owner

Implementata la gestione automatica della versione del manifest e la preparazione delle draft release con lo ZIP scaricabile.

@virtualdj virtualdj self-assigned this May 17, 2024
@moddroid94

Copy link
Copy Markdown
Contributor

Sembra tutto ok, figa la cosa del template!
pero' io ancora non ho capito una cosa: quando lui fai il run dello script python, le modifiche le zippa e le carica nella release, ma nel master? vengono pushate o la versione viene aggiornata solo nello zip?

perche' fa' il checkout all'inizio, quindi clona il master, modifica il suo file manifest locale e zippa, poi carica lo zip, ma le modifiche quando le pusha? e poi nel momento in cui le pusha nel master non triggera un loop in cui parte un altra release? 🤣🤣

dovrebbe tipo pushare le modifiche nella PR e poi fare la release, pero' non so bene se e' giusto cosi

questa cosa secondo me' non funziona come crediamo, ma a scanso di equivoci, faccio che la copio nel mio fork e provo a fare un merge, vediamo cosa esce👌

@virtualdj

Copy link
Copy Markdown
Owner Author

pero' io ancora non ho capito una cosa: quando lui fai il run dello script python, le modifiche le zippa e le carica nella release, ma nel master?

Guardando come funziona il progetto da cui l'ho preso, il master non viene toccato e quindi il manifest lì rimane sempre con la versione 0.0.0 (o quello che c'è scritto nel file).

vengono pushate o la versione viene aggiornata solo nello zip?

Solo nello ZIP. Però poi HACS tira su quello e quindi localmente hai i file corretti.

faccio che la copio nel mio fork e provo a fare un merge, vediamo cosa esce👌

Sì, quello era l'obiettivo 😉 dalle poche prove che ho fatto, a meno di typo, mi pare vada bene. Altra cosa utile usa solo le label per define la nuova versione, senza dover mettere mano alle descrizioni delle commit.

@moddroid94

Copy link
Copy Markdown
Contributor

Ahhhh ecco, infatti non capivo ahahahah che brutta cosa lasciare il master a 0 pero', non sono un grande fan, ma suppongo che funzioni cosi quindi va benissimo, lo copio nel fork 😂

@moddroid94

Copy link
Copy Markdown
Contributor

Allora, ho testato le change e pensavo che la release fosse automatica dopo il merge, usango i commenti, invece lui crea solo una draft, poi qualcuno deve comunque pubblicarla, altrimenti l'action che carica lo zip non si triggera.

se questo e' quello che vogliamo allora sembra funzionare alla grande, le label anche sembrano funzionare.

La aveva anche l'altra action, pero' quella aveva anche la cosa che potevi pushare sul master con [norelease] e non triggerava nulla, pero' appunto generava tutto da sola e comunque pubblicava tutto senza revisione.

PS. anche creare la roba del template per gli issue potrebbe essere una roba carina, almeno si da' qualche indicazione del fatto che un minimo di log potrebbero anche caricarlo 😂

@virtualdj

Copy link
Copy Markdown
Owner Author

che brutta cosa lasciare il master a 0 pero', non sono un grande fan, ma suppongo che funzioni cosi quindi va benissimo, lo copio nel fork 😂

Diciamo che "di là" fanno così, però volendo si potrebbe anche cambiare il master manualmente dato che lo script lo fa automaticamente. Però si perde l'utilità dei tag... Io mi accontenterei, dai 😆

@virtualdj

Copy link
Copy Markdown
Owner Author

invece lui crea solo una draft, poi qualcuno deve comunque pubblicarla, altrimenti l'action che carica lo zip non si triggera

Sì è così, secondo me ha il lato positivo che puoi sistemare il testo delle novità in questa maniera.

se questo e' quello che vogliamo allora sembra funzionare alla grande

Mi sembrava più leggibile della lista delle commit come "novità".

La aveva anche l'altra action, pero' quella aveva anche la cosa che potevi pushare sul master con [norelease] e non triggerava nulla, pero' appunto generava tutto da sola e comunque pubblicava tutto senza revisione.

Io invece non sono un fan del norelease nel testo del commit/merge, mi pare così raggruppi meglio le cose. Poi bisogna provarlo per farsi un'idea più precisa.

PS. anche creare la roba del template per gli issue potrebbe essere una roba carina

Certamente, anche quella è una cosa da fare.

almeno si da' qualche indicazione del fatto che un minimo di log potrebbero anche caricarlo 😂

Tanto non lo caricano comunque 😁 serve un bot che lo ricordi!

@moddroid94

Copy link
Copy Markdown
Contributor

Si no ma in realta' mi piace, meno rischio che vada in prod per sbaglio, tra l'altro funziona anche per le pre-release

per i template degli issue ne avevo trovati di carini nella libreria di lovelace, provo ad aggiungerli a questa PR

per la roba del tenere il manifest a zero bho alla fine considerato che non serve praticamente a nulla direi che possiamo accontentarci dei tag 😂👌

@virtualdj virtualdj marked this pull request as ready for review May 21, 2024 20:31
@virtualdj virtualdj added the minor New minor version label May 21, 2024
@virtualdj virtualdj added the maintenance Applies to changes in actions or dependencies label May 21, 2024
@virtualdj virtualdj merged commit a192675 into master May 21, 2024
@virtualdj virtualdj deleted the feat-github-actions branch May 21, 2024 20:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Applies to changes in actions or dependencies minor New minor version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants