Implement RStatementRangeProvider
#1227
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Joint with posit-dev/ark#85
Part of #1213
Part of #394
This currently has the behavior where if you hit
CMD+Enter
within a function definition (like on line1 + 2
in the video) then it will send the entire function to the console rather than the current line (similar behavior occurs within a function call). RStudio does the opposite, but I actually don't mind this new behavior all that much, and the implementation is very simple for now. You can always opt out of this by doing a manual selection of1 + 2
and sending just that to the consoleScreen.Recording.2023-09-05.at.1.53.44.PM.mov
After this PR (both of which I'll look at):
provideStatementRange()
to allow| undefined
all the way through, rather than catching and erroring on it herepositron/src/vs/workbench/api/common/extHostLanguageFeatures.ts
Lines 1587 to 1593 in 07925f0