terraform icon indicating copy to clipboard operation
terraform copied to clipboard

Fix regex pattern for more alternatives

Open petoju opened this issue 3 years ago • 3 comments

During refactoring, there was a backslash added to | character in terraform regex. That doesn't actually work.

Fix that and return just x|y - that's the correct usage, that works.

Fixes #

Target Release

1.1-backport 1.2-backport 1.3-backport 1.4.x

Draft CHANGELOG entry

BUG FIXES

  • Terraform regex documentation of x|y included an extra backslash. Removed.

petoju avatar Oct 10 '22 15:10 petoju

CLA assistant check
All committers have signed the CLA.

hashicorp-cla avatar Oct 10 '22 15:10 hashicorp-cla

Thanks for the submission! I've notified the docs team. I'm not sure why CLA bot is having issues with this PR. I'll ask around to see if someone knows why that is happening. Thanks!

crw avatar Oct 12 '22 22:10 crw

@crw That was my bad - I used an incorrect user in my commit before.

petoju avatar Oct 14 '22 11:10 petoju

Reminder for the merging maintainer: if this is a user-visible change, please update the changelog on the appropriate release branch.

github-actions[bot] avatar May 05 '23 22:05 github-actions[bot]

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. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

github-actions[bot] avatar Jun 05 '23 02:06 github-actions[bot]