-
Notifications
You must be signed in to change notification settings - Fork 45
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
php-cs-fixer.executablePath works, but not in combination with ${workspaceFolder} that contains a SPACE #216
Comments
@junstyle Is it possible that 2 days ago you released a new version that no longer supports the path filtering? Version where its having troubles loading the config |
@WybeBosch path filtering? what's your config? please paste |
@junstyle My config is still the exact same as before but i was on the newer update. But here is the information you asked for: =========================================================================== This was the issue about quoting the file paths for the config / executeable Path. The below information was when i was still on 0.3.19 Phpcsfixer vscode version: 0.3.19
getting the same error where it doesnt like that there is a space in the root path:
|
Context
#182
I see this previous ticket resulted in making it so that ${workspaceFolder} can be used, which is great
since i have about 100 sites that all have a local version of php-cs-fixer.
For those in the config i use
Which equals to
"/Users/name/Local Sites/client/app/public/vendor/friendsofphp/php-cs-fixer/php-cs-fixer"
The issue
However the issue is that the file path contains a space in the "Local Sites" directory, and yes its stupid that it contains a space, but its not a directory name i can change since im using "Local by Flywheel" which creates and expects the directory to be that way.
Thus resulting in this error:
As you can see it gets confused by the space in the foldername and aborts early.
Is there any way you can modify the way that executablePath is taken in?
Since
I have found this other issue where someone else is having the same issue, and there they modified the internal command the extension uses i believe.
microsoft/vscode#57148
Extra info:
macos m1
PHP 8.1.29 (cli)
Version of php-cs-fixer vscode extension
0.3.14 - 4 days ago (current)
Local version of php-cs-fixer in project "v3.61.1"
Path to local php-cs-fixer
/Users/name/Local Sites/client/app/public/vendor/friendsofphp/php-cs-fixer/php-cs-fixer
Path to local php-cs-fixer config file
/Users/name/Local Sites/client/app/public/.php-cs-fixer.php
config file: https://pastebin.com/9dQuesWu
The text was updated successfully, but these errors were encountered: