Skip to content

Conversation

@Koenkk
Copy link
Owner

@Koenkk Koenkk commented Mar 3, 2025

@Koenkk Koenkk marked this pull request as draft March 3, 2025 21:31
@Nerivec
Copy link
Collaborator

Nerivec commented Mar 4, 2025

Might have to put useNamingConvention it as a warning for ZHC though, lots of variables with snake case that would break ext. converters if renamed (or lots of biome ignore required). MQTT-bound names will also create troubles.
strictCase option will no doubt generate a lot of refactoring (e.g. HTTPServer => HttpServer). Same for the default PascalCase for enum values (following TypeScript maintainers convention).

@Koenkk Koenkk marked this pull request as ready for review March 4, 2025 20:54
@Koenkk
Copy link
Owner Author

Koenkk commented Mar 4, 2025

@Nerivec I added it without making breaking changes

@Koenkk Koenkk merged commit fbf8004 into master Mar 6, 2025
5 checks passed
@Koenkk Koenkk deleted the fix/biome branch March 6, 2025 20:15
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