pun_sensor icon indicating copy to clipboard operation
pun_sensor copied to clipboard

Nuove azioni GitHub

Open virtualdj opened this issue 1 year ago • 7 comments

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

virtualdj avatar May 17 '24 19:05 virtualdj

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👌

moddroid94 avatar May 17 '24 22:05 moddroid94

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.

virtualdj avatar May 18 '24 03:05 virtualdj

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 avatar May 18 '24 09:05 moddroid94

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 😂

moddroid94 avatar May 18 '24 11:05 moddroid94

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 avatar May 18 '24 18:05 virtualdj

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!

virtualdj avatar May 18 '24 18:05 virtualdj

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 😂👌

moddroid94 avatar May 18 '24 23:05 moddroid94