Fix bug with R arguments and incidental multiple matches#3064
Fix bug with R arguments and incidental multiple matches#3064AndreasArvidsson merged 1 commit intomainfrom
Conversation
|
...test? 😄 |
I was a bit hesitant on this one. We already have argument tests for this language. The problem only occurred when you had three or more arguments. I could of course add that test, but right now we have test parity between the languages due to the scope facets. If I add an argument test with three arguments here should I do it for all languages? Or maybe I should just add it here because the Tree sitter looks a bit different from other languages? Should that be a special facet for this and may be other languages? To be honest I'm not sure so I went with the old "When in doubt: do nothing" :D |
|
Hmm good question. Simplest is just to add another test for R with 3 args using the |
|
Sounds reasonable :) |
Both these failed before #3064
Fixes
