virtualdj / pun_sensor

Prezzi PUN del mese - Home Assistant Integration
MIT License
84 stars 15 forks source link

Aggiunte action per il controllo di conformità di HACS #19

Closed mauro-midolo closed 11 months ago

mauro-midolo commented 1 year ago

Ciao @virtualdj

Ho lavorato al codice dell'integrazione e ho apportato alcune modifiche per migliorare la compliance di HACS e di Home Assistant.

Ecco un elenco delle principali modifiche apportate:

  1. Aggiunta di action per il controllo di conformità: Ho introdotto due nuove actions per garantire che il codice sia conforme agli standard del repository e ho corretto gli errori segnalati dalle action preesistenti.

  2. Correzioni delle problematiche principali: Ho effettuato alcune modifiche per migliorare la leggibilità del codice e corrette alcuni dei punti bloccanti.

Azioni da fare dopo l'eventuale merge:

  1. Mancanza dei topics: Purtroppo, non ho i permessi per creare i topics all'interno del repository e una delle richieste di HACS è quella di avere la sezione TOPICS all'interno del repo. ti chiedo quindi di crearla.

Spero che queste modifiche ti aiutino a rendere l'integrazione più robusta e conforme agli standard del progetto. Sarò lieto di discutere ulteriori dettagli o apportare ulteriori modifiche in base ai tuoi feedback.

virtualdj commented 11 months ago

Grazie per il PR, scusa il ritardo ma ho avuto altre cose da fare. Una domanda:

  1. Mancanza dei topics:

Dici he mancano i topics, ma nella pagina di HACS non vedo riferimenti a particolari topics da creare, mi pare di capire che è sufficiente che ce ne siano.

Ebbene, nella mia repo ci sono sempre stati, quindi come mai scrivi che mancano? O forse era solo un appunto da controllare/verificare ma che in realtà era già a posto? Quindi sostanzialmente posso fare il merge così com'è?

mauro-midolo commented 11 months ago

Ciao, era solo un appunto da controllare, perché nel mio fork il job falliva. Se li hai già creati è a posto così.

Puoi fare il merge così com'è, grazie mille

virtualdj commented 11 months ago

Grazie a te!