-
-
Notifications
You must be signed in to change notification settings - Fork 590
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Cannot regex match parentheses after the implementation of string interpolation feature in 4.43.1 #2083
Comments
Hey, yep that's a bug.
That said, I'll fix the bug so when a unclosed interpolation is detected, it will skip interpolation (and log a warning) instead of aborting. |
Is there any usecase where we want string interpolation inside regex matches? I mean, we can always do |
Fixed in 4.44.3 |
This MR contains the following updates: | Package | Update | Change | |---|---|---| | [mikefarah/yq](https://github.com/mikefarah/yq) | patch | `v4.44.2` -> `v4.44.3` | MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot). **Proposed changes to behavior should be submitted there as MRs.** --- ### Release Notes <details> <summary>mikefarah/yq (mikefarah/yq)</summary> ### [`v4.44.3`](https://github.com/mikefarah/yq/releases/tag/v4.44.3) [Compare Source](mikefarah/yq@v4.44.2...v4.44.3) - Fixed upper-case file extension detection, Thanks [@​ryenus](https://github.com/ryenus) ([#​2121](mikefarah/yq#2121)) - Log printing follow no-colors flag [#​2082](mikefarah/yq#2082) - Skip and warn when interpolating strings and theres a unclosed bracket [#​2083](mikefarah/yq#2083) - Fixed CSV content starting with # issue [#​2076](mikefarah/yq#2076) - Bumped dependencies </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this MR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box --- This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNyIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC43IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiXX0=-->
Seems to be working on my end. Thank you |
Describe the bug
When handling regex matching with
yq
, if we want to match against a parentheses, it is not possible. We always get this error about unclosed interpolation stringVersion of yq: >= 4.43.1
Operating system: Linux
Installed via: binary release
Input Yaml
data.yml:
Command
The command you ran:
Actual behavior
Expected behavior
Additional context
I think the string interpolation feature is not working well, regex matching is greatly affected here
The text was updated successfully, but these errors were encountered: