You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
@andycraig I'm playing around with a Windows 10 in virtualbox. I notice there's an environment variable USERPROFILE which happens to be the user home directory. Not sure if we can use this?
@renkun-ken Good find! I tried USERPROFILE it does indeed point to the right location on my Windows PC. But I did some more reading and apparently USERPROFILE and HOMEDRIVE/HOMEPATH don't necessarily point to the same place: https://stackoverflow.com/a/8797855
From os.homedir() docs: 'On Windows, it uses the USERPROFILE environment variable if defined. Otherwise it uses the path to the profile directory of the current user.'
A couple more options:
Rather than having users copy code from README into .Rprofile, provide a vscode-R command that writes the output of os.homedir() to .Rprofile. That would still fail if the user's environment variables changed after that, though.
Look at the source code of os.homedir() and try to duplicate that logic in the code to go in .Rprofile.
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.
Fixes #176
Updates
READMEto give path forinit.Rthat is correct on Windows. (It was already correct on Linux/Mac.)I have tested that this works on Windows and Linux.
@jacob-long I think you are a Windows user too? If you would be able to confirm that this works for you too that would be great.