Refactor config properties to kebab case

LangStream / langstream

LangStream. Event-Driven Developer Platform for Building and Running LLM AI Apps. Powered by Kubernetes and Kafka.

https://langstream.ai

Apache License 2.0

387 stars 28 forks source link

Refactor config properties to kebab case #510

Open nicoloboschi opened 11 months ago

nicoloboschi commented 11 months ago

I’ve noticed different config naming styles: 90% is kebab case (my-property), the rest is pascal (myProperty) and one is with underscore (my_property). We should move them all to the kebab case and keep compatibility

nicoloboschi commented 11 months ago

Some examples:

language-detector uses allowedLanguages
unwrap-key-value uses unwrapKey
text-splitter uses splitter_type, length_function and others
python-* uses className

eolivelli commented 11 months ago

I agree that we should fix all of them as use the kebab case, that is more JSON/YAML friendly.

cbornet commented 11 months ago

💯