Skip to content
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

fix(semanticTokens): make indexing of highlight groups more consistent #4500

Merged
merged 1 commit into from
Feb 3, 2023
Merged

fix(semanticTokens): make indexing of highlight groups more consistent #4500

merged 1 commit into from
Feb 3, 2023

Conversation

Iamnotagenius
Copy link
Contributor

Yeah, remember #4392? Turns out you forgot to change index.ts so when you call semanticTokens.checkCurrent the issue persists.

@codecov
Copy link

codecov bot commented Feb 3, 2023

Codecov Report

Base: 98.76% // Head: 98.76% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (8a7a088) compared to base (bf83393).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4500      +/-   ##
==========================================
- Coverage   98.76%   98.76%   -0.01%     
==========================================
  Files         273      273              
  Lines       25918    25913       -5     
  Branches     5375     5374       -1     
==========================================
- Hits        25599    25593       -6     
- Misses        184      185       +1     
  Partials      135      135              
Impacted Files Coverage Δ
src/core/notifications.ts 98.41% <100.00%> (ø)
src/handler/semanticTokens/index.ts 100.00% <100.00%> (ø)
src/model/notification.ts 97.72% <100.00%> (-2.28%) ⬇️
src/model/progress.ts 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fannheyward fannheyward merged commit 8c0d3ef into neoclide:master Feb 3, 2023
chemzqm added a commit that referenced this pull request Jul 26, 2023
3dc6153 Correct spelling `resutl` to `result` (#4711)
3d14e72 chore(deps): bump word-wrap from 1.2.3 to 1.2.4 (#4706)
2280f9a chore(deps): bump semver from 7.5.0 to 7.5.2 (#4685)
1df60fc docs: fix wrong prefix of CocList command options, ref commit 1d3b424 (#4631)
537f12b fix(rpc): fix unknown function error (#4675)
2a0e554 feat(ci): test with nvim nightly (#4661)
edcc376 chore(README): improve mappings (#4510)
0df7380 fix(notify): use nvim_buf_set_lines for nvim (#4542)
5704b37 chore(exception): typo workspaceContains (#4545)
19c782e chore(yarn): upgrade dev-dependencies (#4627)
e51a7cd fix(list): fix opening an empty buffer in the help action of the extensions (#4652)
3014125 feat: add coc.preferences.tagDefinitionTimeout (#4601)
43f4daa fix(util): properly escape breakat (#4597)
628f08a fix: hide list correctly on vim (#4584)
94dc105 fix(api): Fix Buffer.getVar() for non-current buffer (#4569)
b28b8dc fix(util): patchLine, byte aligned (#4526)
79e9e04 Fix typos and grammatical errors (#4521)
59a942d docs: fix typo termainal -> terminal (#4514)
8c0d3ef fix(semanticTokens): make indexing of highlight groups more consistent (#4500)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants