sassoftware / vscode-sas-extension

This SAS Extension for Visual Studio Code provides support for the SAS language, including features such as SAS syntax highlighting, code completion, hover help, code folding, outline, SAS code snippets and run SAS code.
https://sassoftware.github.io/vscode-sas-extension/
Apache License 2.0
115 stars 47 forks source link

[SSH Connection] Better results file handling #1176

Open ngriffiths21 opened 2 weeks ago

ngriffiths21 commented 2 weeks ago

When using an SSH remote connection with HTML5 results enabled, every SAS submission produces a new .htm file in the home directory on the remote server. In my case, I use the remote server for other activities and want to prevent these files from accumulating (and even if I didn't, it doesn't seem like good practice to fill up the home directory this way).

Describe the solution you'd like It should be possible to configure where the HTML results are stored on the remote server.

Describe alternatives you've considered I tried changing the saspath to a script that first cd's into a custom directory, but this doesn't work because the extension expects the results file in the home directory. The SSH commands used by the extension are not configurable. The only available option appears to be turning off the html5 output, creating a custom "run SAS file" task with preambles and postambles that direct the output to a different location, and loading the results file manually. This is unnecessarily complicated and doesn't make use of the integrated results pane.

Environment SSH remote connection

smorrisj commented 2 weeks ago

For Remote - IOM, we store these ODS HTML5 files in the work directory. This also ensures that they get cleaned up when the session closes. If we were to make the same happen for the Remote - SSH connection method would that work for you?

ngriffiths21 commented 2 weeks ago

Yes, that would be a huge improvement because it would be easy to change with command line options.

smorrisj commented 2 weeks ago

Yes, that would be a huge improvement because it would be easy to change with command line options.

Sounds good. Will try to work this into #1126. @snlwih