-
Notifications
You must be signed in to change notification settings - Fork 53
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
Add Stub File For IDE Autocomplete #30
Comments
Something like this:
|
Good idea. Would you contribute this ? What is the best location to place these stubs ? Maybe phpstorm-stubs directly ? |
RE in phpstorm-stubs directly, I would agree with you but I can't get their tests to pass in master on my machine and I think that's a requirement for them to accept a PR. For reference, I did the same commands as their travis build and was still getting 5 failures each time :/ |
It might be caused by a difference in your environment ? Maybe try to submit a PR anyway |
@arnaud-lb side note, you can also take benefit of PHP 8 new feature, maintain a stub.php and generate arginfo from it (using PHP 8, but usable with older versions). |
This looks convenient, I need to try that |
See or #56 (partial, as aliased functions not handled) |
It would be great if there was a stubfile such as https://github.com/JetBrains/phpstorm-stubs/blob/master/redis/Redis.php
This would make using the extension much more user-friendly
The text was updated successfully, but these errors were encountered: