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

Fix R terminal attachment when the r.rterm setting contains only the executable file name #1545

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

astrochemx
Copy link

@astrochemx astrochemx commented Jul 17, 2024

What problem did you solve?

This PR fixes R terminal attachment when the r.rterm setting contains only the executable file name.
It takes into account if the executable file name already contains the .exe extension in case of the win32 platform.

In addition, this PR:

  • fixes launch.json and changes outDir to dist in all places to fix the extension debugging in VSCode;
  • switches to the Node.js built-in crypto module from the deprecated external one;
  • adds a proper problemMatcher to the npm watch task (the required extension amodio.tsl-problem-matcher is added to the extensions.json);
  • updates settings.json file and fixes formatting in some files;
  • fixes an error message when the extension cannot find the R path.

I would love to hear your thoughts and suggestions on this!

Looking forward to your review and feedback. Thanks!

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.

1 participant