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

#11971 Enhance WinForms Application static class to get current application context #11972

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

CuteLeon
Copy link
Contributor

@CuteLeon CuteLeon commented Aug 23, 2024

Fixes #11971

Proposed changes

  • Add a new public static method on Application: GetCurrentApplicationContext()

Customer Impact

  • A new public API

Regression?

  • No

Risk

  • No clearly risk

Test methodology

  • Can get current application context from another thread
  • Can get current application context from another thread with a message loop

Test environment(s)

  • 9.0.100-preview.7.24407.12
Microsoft Reviewers: Open in CodeFlow

Copy link

codecov bot commented Aug 23, 2024

Codecov Report

Attention: Patch coverage is 97.67442% with 1 line in your changes missing coverage. Please review.

Project coverage is 75.24744%. Comparing base (a9a317d) to head (c3bbbf5).
Report is 75 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #11972         +/-   ##
===================================================
+ Coverage   75.06611%   75.24744%   +0.18133%     
===================================================
  Files           3055        3079         +24     
  Lines         631875      633296       +1421     
  Branches       46776       46824         +48     
===================================================
+ Hits          474324      476539       +2215     
+ Misses        154177      153369        -808     
- Partials        3374        3388         +14     
Flag Coverage Δ
Debug 75.24744% <97.67442%> (+0.18133%) ⬆️
integration 18.00908% <0.00000%> (+0.08181%) ⬆️
production 48.55688% <100.00000%> (+0.33924%) ⬆️
test 97.02099% <97.61905%> (+0.00418%) ⬆️
unit 45.53778% <100.00000%> (+0.27657%) ⬆️

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

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.

Enhance WinForms Application static class to get current application context
1 participant