search

crossplane / terrajet

Generate Crossplane Providers from any Terraform Provider
https://crossplane.io
Apache License 2.0
289 stars 38 forks source link

Consider converting to camel case with acronyms #13

Closed negz closed 2 years ago

negz commented 2 years ago

https://github.com/iancoleman/strcase#custom-acronyms-for-tocamel--tolowercamel

If we're needing to translate snake case Terraform fields to camel case, it may be nice to teach our translation code a few common acronyms so it can capitalize them correctly. That said - in same cases (e.g. AWS) I recall that they don't capitalize acronyms so perhaps it's better to just match their API as closely as possible there. 🤷

muvaf commented 2 years ago

Yes, this was the reason I wanted to use that library. We'll add a conservative init() in terrajet and let provider developer add their own list as well.

muvaf commented 2 years ago

Closing in favor of https://github.com/crossplane-contrib/terrajet/issues/15