Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Show the standard output messages #2343

Merged
merged 1 commit into from
May 29, 2018
Merged

Conversation

akshita31
Copy link
Contributor

Omnisharp-roslyn side : OmniSharp/omnisharp-roslyn#1203

@codecov
Copy link

codecov bot commented May 25, 2018

Codecov Report

Merging #2343 into master will increase coverage by 0.13%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2343      +/-   ##
==========================================
+ Coverage   61.46%   61.59%   +0.13%     
==========================================
  Files          86       86              
  Lines        3851     3859       +8     
  Branches      554      556       +2     
==========================================
+ Hits         2367     2377      +10     
+ Misses       1313     1312       -1     
+ Partials      171      170       -1
Flag Coverage Δ
#integration 51.05% <0%> (-0.06%) ⬇️
#unit 83.45% <100%> (+0.11%) ⬆️
Impacted Files Coverage Δ
src/omnisharp/protocol.ts 78.65% <ø> (ø) ⬆️
src/observers/DotnetTestLoggerObserver.ts 100% <100%> (ø) ⬆️
src/omnisharp/delayTracker.ts 68.42% <0%> (+5.26%) ⬆️

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 0a4c6cc...632e208. Read the comment docs.

@akshita31 akshita31 merged commit 66d72a7 into dotnet:master May 29, 2018
@akshita31 akshita31 deleted the stdout_messages branch May 29, 2018 20:20
@akshita31 akshita31 added this to the 1.16 milestone May 29, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants