-
Notifications
You must be signed in to change notification settings - Fork 9.5k
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
lang/funcs: Add camelCase, kebab-case and snake_case functions #27357
base: main
Are you sure you want to change the base?
Conversation
Codecov Report
|
I would love this @teamterraform :) |
This PR has been languishing for a long time now 😞 Could someone from @teamterraform please comment on whether or not this contribution will be accepted if the conflicts are resolved? |
@teamterraform I am in urgent need of these functions. I was writing logic in my tf module when I saw this PR. |
@teamterraform Can we please have a look at it. This will save hours of work for many of us. Thanks! ❤️ |
return KebabCaseFunc.Call([]cty.Value{str}) | ||
} | ||
|
||
// KebabCase is a Function that converts a given string to kebab-case. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for jumping in, but I just noticed this typo in the comment
// KebabCase is a Function that converts a given string to kebab-case. | |
// CamelCase is a Function that converts a given string to camelCase. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// CamelCase is a Function that converts a given string to camelCase.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed, thank you!
Bumping! Would be nice to have this in terraform :) |
Bumping again! I'm new to TF and this would be great to have. |
This wasn't released yet right? |
This PR is in a holding pattern for now, please see this comment: #28855 (comment), specifically:
Just an FYI as to why this hasn't been reviewed. The conversation is active but there are no updates as to externally-defined functions. Once that has been designed, it will be easier to determine which of the existing requests for functions belong in core versus belong outside of core. |
@teamterraform any update on this? |
@Shrirang97 no updates at this time. |
@crw @teamterraform |
For what it is worth, I am continuing to champion the extendable functions concept. However, there is no update to share. |
Hello, it would really be a plus in the integration of csv files from providers like Google. |
Just FYI, #27696 (comment) |
Is anyone interested in making a provider for these functions? #2771 |
No description provided.