YouCompleteMe
YouCompleteMe copied to clipboard
[WIP/RFC] Report incremental updates with listener_add()
PR Prelude
Thank you for working on YCM! :)
Please complete these steps and check these boxes (by putting an x
inside
the brackets) before filing your PR:
- [x] I have read and understood YCM's CONTRIBUTING document.
- [x] I have read and understood YCM's CODE_OF_CONDUCT document.
- [ ] I have included tests for the changes in my PR. If not, I have included a rationale for why I haven't.
- [x] I understand my PR may be closed if it becomes obvious I didn't actually perform all of these steps.
Why this change is necessary and useful
This is the client side for ycm-core/ycmd#1480
Missing features:
- [x] Don't leak vim hacks into ycmd.
- [x] Find out why are the first two updates full, before switching to incremental updates.
- [x] Undo and redo shouldn't lag behind a change.
[Please explain in detail why the changes in this PR are needed.]
Codecov Report
Merging #3766 (fc3e84e) into master (9ee19df) will decrease coverage by
12.46%
. The diff coverage is92.50%
.
@@ Coverage Diff @@
## master #3766 +/- ##
===========================================
- Coverage 91.29% 78.82% -12.47%
===========================================
Files 27 27
Lines 3711 3741 +30
===========================================
- Hits 3388 2949 -439
- Misses 323 792 +469
Updated to work with ycm-core/ycmd#1506
Seems abandoned. We can always pick this up again in future.