Skip to content

Conversation

@hughbe
Copy link
Contributor

@hughbe hughbe commented May 26, 2019

Fixes #1040
Fixes #1041
Fixes #1045
Fixes #1046
Fixes #1047
Fixes #1048
Fixes #1049
Fixes #1050
Fixes #1051
Fixes #1052

@hughbe hughbe requested a review from a team as a code owner May 26, 2019 23:09
@codecov
Copy link

codecov bot commented May 26, 2019

Codecov Report

Merging #1053 into master will increase coverage by 1.49287%.
The diff coverage is 99.14591%.

@@                 Coverage Diff                 @@
##              master       #1053         +/-   ##
===================================================
+ Coverage   31.31542%   32.80829%   +1.49287%     
===================================================
  Files           1097        1098          +1     
  Lines         318597      320483       +1886     
  Branches       38398       38870        +472     
===================================================
+ Hits           99770      105145       +5375     
+ Misses        214645      211141       -3504     
- Partials        4182        4197         +15
Flag Coverage Δ
#Debug 32.80829% <99.14591%> (+1.49287%) ⬆️
#production 21.44775% <100%> (+1.28705%) ⬆️
#test 98.64512% <99.11504%> (+0.02929%) ⬆️

@hughbe hughbe force-pushed the componentdesigner-test-and-fixes branch from 945ab07 to 6953385 Compare May 29, 2019 19:46
Copy link
Contributor

@zsd4yr zsd4yr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some nits

@JuditRose
Copy link
Contributor

@hughbe Later on, can you please create one PR per one issue?
You can do it either creating one issue for related tasks that can be done in one PR or if there are several issues then create separate PRs for them. It helps us when some of the changes needs to be removed from master for some reason, so they can be reverted separately easily and be tracked by the issue numbers correctly. Thanks!

@JuditRose JuditRose merged commit 1185b1b into dotnet:master May 31, 2019
@hughbe hughbe deleted the componentdesigner-test-and-fixes branch May 31, 2019 17:17
@ghost ghost locked as resolved and limited conversation to collaborators Feb 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.