robertwb commented on PR #31455: URL: https://github.com/apache/beam/pull/31455#issuecomment-2145891819
> Overall, looks really good - my one concern is the clunkiness of string literals... I know that YAML treats everything as a string, so you need some way of emphasizing when an expression should be treated as a string literal, but specifying `'"some string"'` feels messy. Is there a way to modify the `SafeLineLoader` to preserve explicit double quotes? I agree that it's a bit clunky, but I do want the value to be a string that contains quotes in it as a literal. Shouldn't be too common, and it's what I would also use if I was specifying an explicit language. > Something like https://stackoverflow.com/a/67917266 perhaps This seems to be about preserving the quoting style in writing, but it seems dangerous to treat 'possible_var' and "possible_var" differently in parsing. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
