Skip to content

Conversation

@gnikit
Copy link
Member

@gnikit gnikit commented Nov 8, 2022

Fixes #219

@codecov
Copy link

codecov bot commented Nov 8, 2022

Codecov Report

Merging #220 (4462a76) into master (a98765a) will increase coverage by 0.03%.
The diff coverage is 88.88%.

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   86.31%   86.34%   +0.03%     
==========================================
  Files          12       12              
  Lines        4451     4454       +3     
==========================================
+ Hits         3842     3846       +4     
+ Misses        609      608       -1     
Impacted Files Coverage Δ
fortls/langserver.py 83.62% <81.81%> (+0.01%) ⬆️
fortls/objects.py 83.12% <87.50%> (+0.04%) ⬆️
fortls/helper_functions.py 97.73% <100.00%> (ø)
fortls/intrinsics.py 95.68% <100.00%> (+0.12%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Also, added a unittest for intrinsic hover
Default behaviour makes the Markdown hover signature
to be a function of the general hover message
@gnikit gnikit merged commit 513391e into master Nov 9, 2022
@gnikit gnikit deleted the gnikit/issue219 branch November 9, 2022 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

feat: Change autocomplete to use MarkupContent

2 participants