Skip to content

Fix symbol server error handling on older Julia versions#572

Merged
davidanthoff merged 1 commit into
masterfrom
fix-error-handling
Mar 3, 2020
Merged

Fix symbol server error handling on older Julia versions#572
davidanthoff merged 1 commit into
masterfrom
fix-error-handling

Conversation

@davidanthoff

Copy link
Copy Markdown
Member

Fixes an error from crash reporting.

I'm also introducing a new exception type here. The crash reporting can distinguish different different exception types, so this way we don't lump all ErrorExceptions into the same bucket.

@davidanthoff davidanthoff added this to the Current milestone Mar 2, 2020
@davidanthoff
davidanthoff requested a review from ZacLN March 2, 2020 01:14
@codecov

codecov Bot commented Mar 2, 2020

Copy link
Copy Markdown

Codecov Report

Merging #572 into master will increase coverage by 0.06%.
The diff coverage is 80%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #572      +/-   ##
==========================================
+ Coverage   29.87%   29.94%   +0.06%     
==========================================
  Files          22       23       +1     
  Lines        1590     1593       +3     
==========================================
+ Hits          475      477       +2     
- Misses       1115     1116       +1
Impacted Files Coverage Δ
src/LanguageServer.jl 100% <ø> (ø) ⬆️
src/exception_types.jl 100% <100%> (ø)
src/languageserverinstance.jl 47.43% <66.66%> (+0.06%) ⬆️
src/utilities.jl 27.61% <0%> (-0.54%) ⬇️

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 37bbf61...97d3272. Read the comment docs.

@davidanthoff
davidanthoff merged commit 07e7d03 into master Mar 3, 2020
@davidanthoff
davidanthoff deleted the fix-error-handling branch March 3, 2020 19:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants