-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
Feat: Add strcontains function and documentation #33069
Conversation
📄 Content ChecksUpdated: Fri, 21 Apr 2023 15:06:10 GMT Found 1 error(s)
|
Position | Description | Rule |
---|---|---|
1:1-1:1 |
This file is not present in the nav data file at data/language-nav-data.json. Either add a path that maps to this file in the nav data or remove the file. If you want the page to exist but not be linked in the navigation, add a hidden property to the associated nav node. |
no-unlinked-pages |
Thanks. Please add a description for the function here:
That will fix unit tests. |
@Cliftonz Thanks, I'll follow up. |
@Cliftonz apologies for the delay - the docs team is going to try to figure out why CI isn't validating your change. Once that validation passes, we'll merge this PR. Thanks for your patience! |
To be clear, I believe @kmoe will need to merge it. Thanks! |
Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch. |
For posterity, please see #33101 for the follow-up PR that fixes the CI/docs issue. |
* add strcontains function and documentation
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
This PR is a continuation of #27518 and #27198 to complete the function for finding a string with in a string.
This new keyword functions as a contains function in terraform to allow users to find strings instead of using the workaround can(regrex(...)).