Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Linting does not work on Windows 10 #3

Closed
schelhorn opened this issue May 31, 2017 · 4 comments
Closed

Linting does not work on Windows 10 #3

schelhorn opened this issue May 31, 2017 · 4 comments

Comments

@schelhorn
Copy link

When running lintr under Windows 10 with latest VS Code and R extension, the linting does not work on my system since the line (in your extension.js)

    cp.execFile(RPath, parameters, (error, stdout, stderr) => {

does produce an empty stdout. The R command built by this line itself works when run externally, and produces linting messages to stdout. However, when I change the line to

        cp.exec(RPath + " " + parameters.join(" "), (error, stdout, stderr) => {

then linting works just fine in the R extension. Would it be possible to change the line accordingly?

@Ikuyadeu
Copy link
Member

Great! Changing line is possible.
But my develop env is windows 10 and Mac. In my windows, lintr can run.
Could you tell me your settings for R?

@schelhorn
Copy link
Author

Sorry, not required afterall. There issue only appears when I specified Rterm.exe as the interpreter. With R.exe, everything works as intended. Closing.

@Ikuyadeu
Copy link
Member

Ikuyadeu commented Jun 1, 2017

@schelhorn Ok, I should write for windows settings.
Thank you for information.

@Ikuyadeu Ikuyadeu reopened this Jun 1, 2017
@Ikuyadeu Ikuyadeu closed this as completed Jun 1, 2017
@wedneyyuri
Copy link
Contributor

wedneyyuri commented Nov 15, 2017

Hi @Ikuyadeu,

I'm using Windows 10, vscode 1.18.0 and R (version 3.4.2 (2017-09-28) -- "Short Summer") but the Lintr is not working for me. It's a fresh installation, so I don't know if I did anything wrong.

User Settings:

{ "r.rterm.windows": "C:\\Program Files\\R\\R-3.4.2\\bin\\x64\\R.exe" }

image

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

No branches or pull requests

3 participants