Skip to content

Conversation

@Nora-Zhou01
Copy link
Contributor

@Nora-Zhou01 Nora-Zhou01 commented Jul 25, 2024

Related #10773

Proposed changes

  • Add unit test DesignBindingTests.cs for public properties and method of the DesignBinding.
  • Enable nullability in DesignBinding.
Microsoft Reviewers: Open in CodeFlow

@Nora-Zhou01 Nora-Zhou01 requested a review from a team as a code owner July 25, 2024 01:44
@codecov
Copy link

codecov bot commented Jul 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.76861%. Comparing base (6a243b0) to head (08997aa).
Report is 41 commits behind head on feature/10.0.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           feature/10.0      #11748         +/-   ##
======================================================
+ Coverage      74.68804%   74.76861%   +0.08056%     
======================================================
  Files              3045        3054          +9     
  Lines            631338      631174        -164     
  Branches          47177       46880        -297     
======================================================
+ Hits             471534      471920        +386     
+ Misses           156443      155883        -560     
- Partials           3361        3371         +10     
Flag Coverage Δ
Debug 74.76861% <100.00000%> (+0.08056%) ⬆️
integration 17.86714% <ø> (-0.05726%) ⬇️
production 47.78461% <ø> (+0.07186%) ⬆️
test 96.98826% <100.00000%> (+0.01004%) ⬆️
unit 44.91929% <ø> (+0.18081%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Contributor

@Tanya-Solyanik Tanya-Solyanik left a comment

Choose a reason for hiding this comment

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

Looks good, I added a couple of code-style comments

@Tanya-Solyanik Tanya-Solyanik added the waiting-author-feedback The team requires more information from the author label Jul 26, 2024
@dotnet-policy-service dotnet-policy-service bot removed the waiting-author-feedback The team requires more information from the author label Jul 29, 2024
Copy link
Member

@lonitra lonitra left a comment

Choose a reason for hiding this comment

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

Thank you

@lonitra lonitra merged commit c947cc9 into dotnet:feature/10.0 Jul 30, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Aug 30, 2024
@Nora-Zhou01 Nora-Zhou01 deleted the Add-core-coverage-for-DesignBinding branch September 30, 2024 06:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants