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

fixed #11214 #11213

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

swapso
Copy link

@swapso swapso commented Apr 14, 2024

Fixes #11214

Link for issue: #11214

Proposed changes

  • Updated if condition to bypass base.IsHandleCreated

Regression?

  • No
Microsoft Reviewers: Open in CodeFlow

@swapso swapso requested a review from a team as a code owner April 14, 2024 10:20
Copy link

codecov bot commented Apr 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.47240%. Comparing base (9c9b2e0) to head (4ec1707).
Report is 117 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                 @@
##                main      #11213         +/-   ##
===================================================
+ Coverage   73.39203%   73.47240%   +0.08036%     
===================================================
  Files           3097        3101          +4     
  Lines         632134      632447        +313     
  Branches       46650       46688         +38     
===================================================
+ Hits          463936      464674        +738     
+ Misses        164802      164366        -436     
- Partials        3396        3407         +11     
Flag Coverage Δ
Debug 73.47240% <100.00000%> (+0.08036%) ⬆️
integration 18.36704% <100.00000%> (+0.15758%) ⬆️
production 46.96004% <100.00000%> (+0.15935%) ⬆️
test 94.98827% <ø> (+0.00560%) ⬆️
unit 43.93314% <100.00000%> (+0.05272%) ⬆️

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

@elachlan
Copy link
Contributor

@swapso can you update the PR description with the correct issue ID?

@elachlan elachlan added the 📭 waiting-author-feedback The team requires more information from the author label Apr 21, 2024
@swapso swapso changed the title fixed #101019 fixed #11214 Apr 24, 2024
@dotnet-policy-service dotnet-policy-service bot removed the 📭 waiting-author-feedback The team requires more information from the author label Apr 24, 2024
@swapso
Copy link
Author

swapso commented Apr 24, 2024

@swapso can you update the PR description with the correct issue ID?

Hi @elachlan I have updated correct Issue ID.

@elachlan elachlan added waiting-review This item is waiting on review by one or more members of team waiting-on-team This work item needs to be discussed with team or is waiting on team action in order to proceed labels Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-on-team This work item needs to be discussed with team or is waiting on team action in order to proceed waiting-review This item is waiting on review by one or more members of team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Font autoscaling doesn't function in .NET Core as it does in .NET Framework 4.8.
2 participants