psalm-vscode-plugin icon indicating copy to clipboard operation
psalm-vscode-plugin copied to clipboard

Psalm not able to use psalm.phpExecutablePath setting in VS Code, uses default php instead

Open matlinski opened this issue 1 year ago • 4 comments

Hello, I never write git issues so forgive me if it's not well structured.

I have a WSL set up in my windows and there are multiple project inside. Most need PHP 7.4 and that's how default php is configured. There is one project where I need to run php8.1. In that project there is no way to set the alternative binary path for psalm. I tried introducing the following changes to workspace.code-workspace:

{
   "settings": {
		"php.validate.executablePath": "/usr/bin/php8.1",
		"psalm.phpExecutablePath": "/usr/bin/php8.1"
   }
}

the first path solved the issue for default VS code validation, the second path was added automatically by from VS code Psalm extension GUI (see the screenshot). photo_2023-03-06_14-34-28

This should defitelly solve the problem but it didn't. Both VS code extension and composer extension are up to date.

matlinski avatar Mar 06 '23 19:03 matlinski

This is working fine for me.

Note that I went and set: "psalm.phpExecutablePath": "/usr/bin/ppp" which is invalid and I got an error message that its not executable. I then set it to /dev/null which worked as well

tm1000 avatar Mar 07 '23 22:03 tm1000

I believe the desired effect is in a multi-root workspace, config settings which are relevant to Psalm should have a scope of resource or machine-overridable instead of window.

In other words, every member of workspace.workspaceFolders should spawn an individual LanguageServer.ts, run an individual vendor/bin/psalm process.

I have a sample implementation of how this might work in https://github.com/limarkxx/psalm-vscode-plugin/compare/integration-testing...limarkxx:psalm-vscode-plugin:folder-level-settings?diff=split

limarkxx avatar Jul 24 '23 20:07 limarkxx

@limarkxx Would you be willing to push that as a PR

tm1000 avatar Jul 24 '23 22:07 tm1000

@tm1000 I've set https://github.com/psalm/psalm-vscode-plugin/pull/268 as a draft PR

limarkxx avatar Jul 24 '23 22:07 limarkxx