YouCompleteMe icon indicating copy to clipboard operation
YouCompleteMe copied to clipboard

[WIP/RFC] Report incremental updates with listener_add()

Open bstaletic opened this issue 3 years ago • 2 comments

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.]


This change is Reviewable

bstaletic avatar Sep 22 '20 07:09 bstaletic

Codecov Report

Merging #3766 (fc3e84e) into master (9ee19df) will decrease coverage by 12.46%. The diff coverage is 92.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     

codecov[bot] avatar Sep 22 '20 07:09 codecov[bot]

Updated to work with ycm-core/ycmd#1506

bstaletic avatar Nov 23 '20 15:11 bstaletic

Seems abandoned. We can always pick this up again in future.

puremourning avatar Sep 29 '22 19:09 puremourning