-
Notifications
You must be signed in to change notification settings - Fork 185
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
LanguageServer script failing #795
Comments
Same. My "workaround" was to simply remove |
same err here 👎 |
You can use the bundled language server from the vscode extension https://marketplace.visualstudio.com/items?itemName=felixfbecker.php-intellisense --> download extension --> rename to .zip --> /extension/vendor/felixfbecker/language-server/bin |
Also fails on vendor/jetbrains/phpstorm-stubs/standard/standard_8.php |
phpstorm-stubs dev-master already had PHP8 features which are breaking the parser per felixfbecker#795. The simple solution for this is to force an older version of the stubs.
I submitted a simple pull request to fix this. The simple solution until that's merged is to setup your composer.json like so: {
"repositories": [
{
"type": "vcs",
"url": "https://github.com/CodyWilson/php-language-server"
}
],
"require": {
"felixfbecker/language-server": "dev-master"
}
} Alternatively you could fork it yourself and do that change, and set the version control to point to your fork. |
@felixfbecker any updates for this project? |
Guys/ladies, I'm having some trouble getting composer/language-server setup in Ubuntu Linux. This is the latest error in my quest to finish this apparently insurmountably difficult task. Granted I'm new to composer, so this is all still a bit confusing.
After running
composer run-script --working-dir=vendor/felixfbecker/language-server parse-stubs
The script fails on line
Parsing file:///home/phatwhalemedia/.config/composer/vendor/jetbrains/phpstorm-stubs/http/http.php
Giving this error
If there's any further info required, please let me know.
I've been following the steps from the following link:
#611
The text was updated successfully, but these errors were encountered: