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

Expose CreateExceptionFromRpcError as a protected virtual method #518

Merged
merged 1 commit into from
Aug 11, 2020

Conversation

AArnott
Copy link
Member

@AArnott AArnott commented Aug 10, 2020

This provides the counterpart to the pre-existing virtual method CreateErrorDetails.

Closes #502

This provides the counterpart to the pre-existing virtual method `CreateErrorDetails`.

Closes microsoft#502
@AArnott AArnott added this to the v2.6 milestone Aug 10, 2020
@AArnott AArnott requested review from javierdlg and milopezc August 10, 2020 19:51
@AArnott AArnott self-assigned this Aug 10, 2020
@codecov-commenter
Copy link

codecov-commenter commented Aug 10, 2020

Codecov Report

Merging #518 into master will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #518   +/-   ##
=======================================
  Coverage   89.12%   89.13%           
=======================================
  Files          51       51           
  Lines        4065     4067    +2     
=======================================
+ Hits         3623     3625    +2     
  Misses        442      442           
Impacted Files Coverage Δ
src/StreamJsonRpc/JsonRpc.cs 92.43% <100.00%> (+0.01%) ⬆️

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 e99a08c...f5d69f9. Read the comment docs.

@AArnott AArnott merged commit 6a19eae into microsoft:master Aug 11, 2020
@AArnott AArnott deleted the fix502 branch August 11, 2020 19:21
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.

Expose JsonRpc.CreateExceptionFromRpcError as a protected virtual method
3 participants