search

dyaliCode / svelte-formly

Generator dynamic forms for Svelte JS
https://svelte.formly-js.com
MIT License
253 stars 30 forks source link

Remove Snake Case use #59

Open Lucasmiguelmac opened 2 years ago

Lucasmiguelmac commented 2 years ago

Hello there.

I've seen a bad pattern here, and it is that snake_case is being used in this library. This is a total bad practice and breaks patterns that are a standard across JS/TS libraries.

It may be ok to use snake_case with other programming languages (ie: Python), but when working with TypeScript or JavaScript, we should stick to using camelCase, or PascalCase when naming components and objects.

Take a look at any other JS/TS/Svelte library and you'll quickly notice that snake_case is not used anywhere.

kamalkech commented 2 years ago

@Lucasmiguelmac thank for ur investigation, if u look to react version i clean the code and i will do the same with svelte/solid versions, it s better to join to my discord if u have any questions + ideas ;) https://discord.gg/cAjxEsHe