-
Notifications
You must be signed in to change notification settings - Fork 185
Cache project index, not just dependencies #578
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
Comments
@geromueller the LS uses FileSystemCache as default. Anything that indicates to you that it doesn't? |
@felixfbecker I replied via Email, but only to you, so here are more details: Every time I open VSC it re-indexes, no cache directory present.
still no cached files.. The project consists of nested PHP projects (composer.json/.lock).. |
Bump! |
What is your log output in the output panel? |
[Info - 10:11:27] 9489 files total |
Have one workspace with 4 folders, each containing a composer project |
Only dependencies are cached because we know to invalidate the cache if the version changed. How would you invalidate the project cache? Any changes can happen between runs, like a git checkout |
On startup you could compare the ctime of the cache file and the source file. And during runtime use inotify? |
My usual Drupal project has 18766 files total which takes even longer. Is there some workaround better than disabling intellisence? |
The current workaround is to use the intelephense extension https://marketplace.visualstudio.com/items?itemName=bmewburn.vscode-intelephense-client instead, which is also recommended by the drupal.org VSCode documentation at https://www.drupal.org/docs/develop/development-tools/configuring-visual-studio-code |
I can confirm that the project index is basically rebuilt every time things are restarted, which makes this project mostly worthless to me. Would be nice to have indexing saved. |
I work on a huge PHP project and indexing takes ~30minutes. Currently PHPLS uses the xcache caching, rendering the extension useless for me.
FileSystemCache would make a better default I think, and only use xache on demand.
The text was updated successfully, but these errors were encountered: