Skip to content
This repository was archived by the owner on Nov 25, 2021. It is now read-only.

Conversation

@ijsnow
Copy link
Contributor

@ijsnow ijsnow commented Aug 17, 2018

The python language server responds with contents but not a range sometimes. This means that we can end up with tooltip data and the tooltip displayed but no token highlighting. This PR rearranges some stuff to more robustly handle this and still adjust the position each time if necessary.

@codecov
Copy link

codecov bot commented Aug 17, 2018

Codecov Report

Merging #45 into master will decrease coverage by 1.1%.
The diff coverage is 90%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
- Coverage   66.08%   64.98%   -1.11%     
==========================================
  Files          15       15              
  Lines         631      634       +3     
  Branches      165      169       +4     
==========================================
- Hits          417      412       -5     
- Misses        214      222       +8
Impacted Files Coverage Δ
src/hoverifier.ts 55.43% <90%> (-3.13%) ⬇️
src/helpers.ts 53.84% <0%> (-2.57%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2ed1850...94c2999. Read the comment docs.

@ijsnow ijsnow merged commit ab332f9 into master Aug 17, 2018
@sourcegraph-bot
Copy link

🎉 This PR is included in version 3.7.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

@ijsnow ijsnow deleted the adjust-pos-fix branch August 17, 2018 23:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants