Skip to content

Conversation

@Philip-Wang01
Copy link
Contributor

@Philip-Wang01 Philip-Wang01 commented Aug 27, 2024

Related #10773

Proposed changes

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

@codecov
Copy link

codecov bot commented Aug 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.07009%. Comparing base (3b37d24) to head (ac6fcaa).
Report is 3 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #11989         +/-   ##
===================================================
+ Coverage   75.06780%   75.07009%   +0.00228%     
===================================================
  Files           3059        3060          +1     
  Lines         632006      632044         +38     
  Branches       46782       46782                 
===================================================
+ Hits          474433      474476         +43     
  Misses        154192      154192                 
+ Partials        3381        3376          -5     
Flag Coverage Δ
Debug 75.07009% <100.00000%> (+0.00228%) ⬆️
integration 17.93071% <ø> (+0.00388%) ⬆️
production 48.22234% <ø> (+0.00175%) ⬆️
test 97.01666% <100.00000%> (+0.00032%) ⬆️
unit 45.25484% <ø> (ø)

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.

Thank you!

@lonitra lonitra merged commit c9ca728 into dotnet:main Aug 27, 2024
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0 Preview1 milestone Aug 27, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Sep 28, 2024
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.

3 participants