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.

Sign up for GitHub

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

Jump to bottom

Improve did-change performance #334

Merged

ericdallo merged 2 commits into master from improve-did-change

Feb 21, 2021

Member

ericdallo commented Feb 21, 2021

Add flag to disable/enable notify-references
Disable notify-references by default for now, needs hammock time
Make did-change completely async as we publish diagnostics later after scan


          Disable notify-references for now

e8ed41f

ericdallo self-assigned this

ericdallo added the performance label


          Use async from main as didChange is a notification

8016ad7

ericdallo force-pushed the improve-did-change branch from f5c6310 to 8016ad7 Compare

February 21, 2021 17:11

ericdallo marked this pull request as ready for review

February 21, 2021 17:17

ericdallo merged commit e1e3d0c into master

ericdallo deleted the improve-did-change branch

February 21, 2021 17:17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment