Skip to content

Conversation

@liam923
Copy link
Contributor

@liam923 liam923 commented Feb 25, 2025

This PR goes along with ocaml/merlin#1885, which changes Merlin's api for occurrences requests to include more staleness information.

@coveralls
Copy link

coveralls commented Feb 28, 2025

Pull Request Test Coverage Report for Build 4784

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 19 (0.0%) changed or added relevant lines in 2 files are covered.
  • 40 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.8%) to 22.972%

Changes Missing Coverage Covered Lines Changed/Added Lines %
ocaml-lsp-server/src/rename.ml 0 5 0.0%
ocaml-lsp-server/src/ocaml_lsp_server.ml 0 14 0.0%
Files with Coverage Reduction New Missed Lines %
ocaml-lsp-server/src/document_symbol.ml 40 41.58%
Totals Coverage Status
Change from base Build 4763: 0.8%
Covered Lines: 5872
Relevant Lines: 25562

💛 - Coveralls

@voodoos voodoos force-pushed the stale-occurrences branch from 92f1a8c to 4874941 Compare March 4, 2025 14:54
@voodoos
Copy link
Collaborator

voodoos commented Mar 4, 2025

Thanks Liam!

@voodoos voodoos merged commit dba52c2 into ocaml:master Mar 4, 2025
6 checks passed
@liam923 liam923 deleted the stale-occurrences branch March 5, 2025 15:03
@liam923 liam923 restored the stale-occurrences branch March 5, 2025 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants