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

[IT] Update Common #2134

Merged
merged 22 commits into from
Apr 15, 2024
Merged

[IT] Update Common #2134

merged 22 commits into from
Apr 15, 2024

Conversation

malorezzanzo64
Copy link
Contributor

Added floor, reorganized a bit, corrected some errors and added some other stuff

malorezzanzo64 and others added 10 commits April 12, 2024 19:20
Aggiunta lista brightness_level, aggiunte regole di espansione some, which, lock, to_lock e ampliate altre liste ed regole di espansione
remove duplicate brightness_level
@auanasgheps
Copy link

Ciao, vedo che hai lavorato un sacco stanotte, ma la verifica automatica fallisce senza mostrare un errore e non saprei da dove iniziare. Considera però il commit "added floor" ha passato i controlli, forse possiamo ripartire da lì.

@malorezzanzo64
Copy link
Contributor Author

Ciao, si stanotte è stata lunga.😆 Ho notato che dava errore ma come te non sapevo da dove iniziare. Purtroppo il motivo per cui "added floor" ha passato i controlli mi pare essere che Github non ha avuto il tempo di fare entrambi i controlli prima che facessi un nuovo edit😢

@malorezzanzo64
Copy link
Contributor Author

@auanasgheps alla fine ce l'ho fatta!! 🎉
Era più banale del previsto. L'errore che dava diceva semplicemente che il file aveva più di una riga vuota consecutiva, e quindi non era "bellissimo". Prettier toglieva queste righe in più ma dato che prettier modificava il file dava errore.
Poi ci stavano altri errori nello script tests che dipendevano principalmente dal fatto che le regole di espansione di , , vanno attaccate alla parola seguente, perche altrimenti per esempio " interruttore" non matcha "l'interruttore".

@malorezzanzo64
Copy link
Contributor Author

Comunque mi ha dato una grandissima mano il codespace e fare i test nel codespace perchè potevo fare una piccola modifica al codice e vedere che errori dava senza ogni volta fare commit

@auanasgheps
Copy link

@malorezzanzo64 non ci credo, una riga vuota consecutiva? 😂 Alla faccia dei controlli automatici, poveri noi...

Figata la funzione del codespace, non conoscevo!

Rivedo le modifiche per eventuali errori di battitura e se è tutto ok faccio il merge, chiudendo finalmente questa impresa!

@auanasgheps auanasgheps merged commit 8e13ede into home-assistant:main Apr 15, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants