Skip to content

Conversation

@cpsievert
Copy link
Contributor

@cpsievert cpsievert commented Jul 31, 2020

See #401 for motivation.

This disables completion of R code inside quotes (when exercise.completion = TRUE), by setting quotes = FALSE in rc.settings()...the docs for quotes says:

Enables completion in R code when inside quotes. This normally leads to filename completion, but can be otherwise depending on context (for example, when the open quote is preceded by ?), help completion is invoked. Setting this to FALSE relegates completion to the underlying completion front-end, which may do its own processing (for example, readline on Unix-alikes will do filename completion).

Thus, this might be a slightly over-agressive way to prevent filenames from being displayed. Note that there is also a files argument that we could set to FALSE, but for some reason that arg is deprecated (maybe it's effectively the same as quotes?)

@cpsievert cpsievert changed the title Disbale completion of R code inside quotes, closes #401 Disable completion of R code inside quotes, closes #401 Jul 31, 2020
@cpsievert cpsievert requested a review from schloerke August 3, 2020 14:50
Copy link
Collaborator

@schloerke schloerke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM given suggestion!

Co-authored-by: Barret Schloerke <barret@rstudio.com>
@cpsievert cpsievert merged commit f088958 into master Aug 4, 2020
@cpsievert cpsievert deleted the carson/filename-completion branch August 4, 2020 15:54
schloerke added a commit that referenced this pull request Aug 7, 2020
* master:
  Disable completion of R code inside quotes, closes #401 (#413)
  Bump version (#414)
  Save tutorial output to temp folder when tutorial folder does not have write permissions (#412)
  Throw an informative error if an exercise chunk is NULL (#411)
  Remove outdated exercise option
  Checking for exercise errors (#403)
schloerke added a commit that referenced this pull request Sep 2, 2020
* master:
  Add events to website (#423)
  Add a checker test with error as solution (#422)
  Fixes for learnr on external evaluator (#420)
  Add shinytest support (#407)
  Validate the ability to create an idb store (#417)
  Disable completion of R code inside quotes, closes #401 (#413)
  Bump version (#414)
  Save tutorial output to temp folder when tutorial folder does not have write permissions (#412)
  Throw an informative error if an exercise chunk is NULL (#411)
  Remove outdated exercise option
  Checking for exercise errors (#403)
  Add event handler system (#398)
  External evaluator fixes (#399)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants