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

Initial support for textDocument/hover request. #2087

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Initial support for textDocument/hover request. #2087

merged 1 commit into from
Feb 23, 2024

Conversation

hzeller
Copy link
Collaborator

@hzeller hzeller commented Jan 30, 2024

Rebased from PR #1922

This will implement the method, but currently it is not offered yet in the initialization to the editor; to be enabled once tests finished.

@hzeller
Copy link
Collaborator Author

hzeller commented Jan 30, 2024

Hi Grzegorz, I moved your the draft pull request here, so that the work is not going out-of-sync and it is easier to work on it later.
Had to fix a few conflicts, but tests are now working :)

@hzeller hzeller requested a review from fangism January 30, 2024 07:44
@codecov-commenter
Copy link

codecov-commenter commented Jan 30, 2024

Codecov Report

Attention: Patch coverage is 96.84211% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 92.92%. Comparing base (0f97047) to head (65a6aec).

❗ Current head 65a6aec differs from pull request most recent head 2d1dd11. Consider uploading reports for the commit 2d1dd11 to get more accurate results

Files Patch % Lines
verilog/tools/ls/hover.cc 96.15% 3 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2087   +/-   ##
=======================================
  Coverage   92.92%   92.92%           
=======================================
  Files         358      359    +1     
  Lines       26577    26666   +89     
=======================================
+ Hits        24696    24780   +84     
- Misses       1881     1886    +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Rebased from PR #1922

This will implement the method, but currently it is not offered
yet in the initialization to the editor; to be enabled once tests
finished. The implementation requires a global symbol table, which
is potentially a bit heavy. We might need to introduce the concept of
a per-file symbol table for faster local lookups.
@hzeller hzeller merged commit 528d284 into chipsalliance:master Feb 23, 2024
30 of 31 checks passed
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.

3 participants