virtualdj / pun_sensor

Prezzi PUN del mese - Home Assistant Integration
MIT License
78 stars 13 forks source link

Nuove azioni GitHub #46

Closed virtualdj closed 1 month ago

virtualdj commented 1 month ago

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

moddroid94 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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 commented 1 month ago

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