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

settings: simplify flow to set path for an executable #89134

Closed
ghost opened this issue Jan 23, 2020 · 2 comments
Closed

settings: simplify flow to set path for an executable #89134

ghost opened this issue Jan 23, 2020 · 2 comments
Assignees
Labels
php PHP support issues

Comments

@ghost
Copy link

ghost commented Jan 23, 2020

Please search existing issues to avoid creating duplicates.

searched lots of random issues.
example: #251 #22391 #25576


I encountered this for php, but should be general.

  1. copy-pasted from my path to appease 'PHP Language features (Extension)'
    "php.validate.executablePath":"C:\Users\username\Downloads\php-7.4.2"

expected it to work.

Describe the feature you'd like

It can't understand it.

if it can parse the string correctly

two better ways to improve the situation would be

  • search for php file in the given folder, since the value(string) is given.
    Or
  • name the key to be more intuitive. one example: - "php.validate.pathToExecutable"

there's more than two issues here. My end-goal is to make it straightforward to remove such vscode pop-ups without having to dabble in different things for not much benefit. Starting with php as its not the main focus of vscode, so issues should be minimal.

Don't have a statistic or anything to point to as indication that its going to benefit vscode users. Just one more thing I personally wanted to do in vscode and hit a speed-bump.

@vscodebot vscodebot bot added the php PHP support issues label Jan 23, 2020
@ghost
Copy link
Author

ghost commented Jan 23, 2020

way to ignore issue and accomplish what vscode's 'PHP Language features (Extension)' wants:
"php.validate.executablePath":"C:\\Users\\username\\Downloads\\php-7.4.2"

this will allow it to interpret it. now it will display a false warning.

using
C:\\Users\\username\\Downloads\\php-7.4.2\\php.exe
should silence it. now you can go back to doing what you want.


you can also disable the PHP Language features (Extension) I realised from looking into it that its the sane default for builtin extensions of this type, if builtin programming language related extensions are not a big issue. Syntax highlighting still works, using built-in programming language extensions ending in 'Basics'

Disabling them should also improve performance. Definitely a better experience without all silly pop-ups. Wish there was a way to disable builtin programming language extensions easily instead of one-by-one.

I will add this to my wiki. feel free to do the same, if it suits the project needs.

@roblourens
Copy link
Member

Yeah the backslashes need to be escaped, that's just JSON's rules. You can also use forward slashes.

@github-actions github-actions bot locked and limited conversation to collaborators Jun 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
php PHP support issues
Projects
None yet
Development

No branches or pull requests

1 participant