Skip to content

Implementazione DevContainer per VSCode#55

Merged
virtualdj merged 4 commits into
masterfrom
devcontainer
Oct 26, 2024
Merged

Implementazione DevContainer per VSCode#55
virtualdj merged 4 commits into
masterfrom
devcontainer

Conversation

@virtualdj

Copy link
Copy Markdown
Owner

Questa PR dovrebbe implementare un DevContainer per VS Code che permetta:

  • il debug dell'integrazione pun_sensor con la più recente versione di HA senza dover clonare la sua repository (che sarebbe il metodo "ufficiale")
  • il linting con Ruff
  • la possibilità di effettuare direttamente le commit sul repository git clonato

Necessità nata da qui:

Mi sono impuntato con questa storia dei DevContainer (infatti ho postato anche sulla community di HA se hai visto) perché vorrei proprio trovare una soluzione migliore alla doppia istanza di VS Code. Ma ancora non ho cavato un 🕷 dal buco.

@moddroid94 Quando hai un attimo, daresti un'occhiata al nuovo branch devcontainer? Dopo enormi sacrifici (ammetto di non averne capito molto) credo di essere giunto ad un buon punto pescando di qua e di là (sì, nessuno aveva qualcosa di già pronto che mi soddisfasse).

Vorrei capire se anche nel tuo sistema tale devcontainer funziona correttamente... dovrebbe andare sia il debug (F5) che il task di esecuzione di HA (F1) se non si ha necessità di debugging ma solo di leggere i log.

Domande:

  • Ho implementato tutto corretamente?
  • Funziona il linting automatico al salvataggio di un file?
  • Per ora ho commentato il pre_commit perché credo che faccia più danni che altro, non so se sia la scelta giusta.
  • Vale la pena modificare le action di GitHub per fare un controllo con Ruff? Sono in dubbio anche qui e non sono sicuro di come impostarlo correttamente.

@virtualdj virtualdj added the maintenance Applies to changes in actions or dependencies label Aug 10, 2024
@virtualdj virtualdj requested a review from moddroid94 August 10, 2024 12:15
@virtualdj virtualdj merged commit fd7bdce into master Oct 26, 2024
@virtualdj virtualdj deleted the devcontainer branch October 26, 2024 15:17
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant