Kris Shannon

Hi, I've updated to master and formatted the commit message according to the guidelines. I don't actually think this is really the right solution, it's basically just the hack we're...

I'm guessing my drop and run commit against stable confused frrbot. I think the only labels which are really relevant are `bgp` and `vtysh`

This really needs a solution of some kind in the core. Workarounds like the `command("find ...")` are not general enough. It probably needs work in the train library

As a workaround we are adding this to the themes `functions.php` of any affected site: ```php /** * Fix ESI for Litespeed Cache and Modern Maven theme */ remove_action('wp_body_open', 'wp_admin_bar_render',...

Separating the name into namespace, subsystem, and name worked. If this is now required, then the docs need updating. From [Telemetry >> Collector configuration](https://www.joyent.com/containerpilot/docs/configuration/telemetry#collector-configuration): > You can leave off the...

I don't want to put words into @archoversight's mouth, but I think the problem they are trying to solve is moving a `for_each` from a resource in the parent module...

It may make sense to extract the creation of the string Replacer outside of the `stringValue` function and use a single Replacer for all literals instead of a new one...

Just downloaded the new v4.34.1 and tested with it. Appears to be the same behaviour

This appears to be the [lexing of the literal strings](https://github.com/mikefarah/yq/blob/v4.34.1/pkg/yqlib/lexer_participle.go#L365-L375): ```golang func stringValue() yqAction { return func(rawToken lexer.Token) (*token, error) { log.Debug("rawTokenvalue: %v", rawToken.Value) value := unwrap(rawToken.Value) log.Debug("unwrapped: %v", value)...

Created a pull request #1814 with a possible fix.