search

lf-lang / vscode-lingua-franca

Lingua Franca extension for Visual Studio Code
Other
5 stars 3 forks source link

Updated message for missing dependencies #123

Closed petervdonovan closed 1 year ago

petervdonovan commented 1 year ago

This is mostly relevant to packages managed by NVM, which seem to be located in a special place.

Fixes #119. This does not actually fix the PATH, but it does at least update the error message so that the message suggests the correct solution and is not misleading.

I believe that we can fix the PATH if we really want to by doing some hacking with the embedded terminal. However, I anticipate that such a solution would cause bugs and maintainability problems that I do not want to face right now.

petervdonovan commented 1 year ago

It was my mistake to open this PR without waiting for CI to pass. I think it is truly ready to merge now.