These are nice tree-sitter features but are a bit annoying for us. It means that DocumentContext will drill allllllll the way down to one of the following, rather than finding and stopping at just a NodeType::String like before:
Anonymous("'") or Anonymous("\"")
StringContent
EscapeSequence
This means we have to walk back up the tree a bit when we need to work with or detect the String node. I think I've found all the places we do this.
I'm hoping this is the most annoying tree-sitter related tweak I'll have to do.
Bumps us one more commit in tree-sitter-r to support the new
string_contentandescape_sequencechildren of astringnode https://github.com/r-lib/tree-sitter-r/compare/bc8919d3c38b816652e1e2d1a1be037cf74364cb...9d1a68f8f239bc3749a481ac85e2163e24f6362c.These are nice tree-sitter features but are a bit annoying for us. It means that
DocumentContextwill drill allllllll the way down to one of the following, rather than finding and stopping at just aNodeType::Stringlike before:Anonymous("'")orAnonymous("\"")StringContentEscapeSequenceThis means we have to walk back up the tree a bit when we need to work with or detect the
Stringnode. I think I've found all the places we do this.I'm hoping this is the most annoying tree-sitter related tweak I'll have to do.