Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implementazione DevContainer per VSCode #55

Merged
merged 4 commits into from
Oct 26, 2024
Merged

Implementazione DevContainer per VSCode #55

merged 4 commits into from
Oct 26, 2024

Conversation

virtualdj
Copy link
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 merged commit fd7bdce into master Oct 26, 2024
2 checks passed
@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