Skip to content

Flip nullable enable on System.Windows.Forms.Tests. #13063

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

Merged
merged 3 commits into from
Mar 4, 2025

Conversation

JeremyKuhne
Copy link
Member

@JeremyKuhne JeremyKuhne commented Mar 3, 2025

Created a simple script using ChatGPT that did most of the lifting. There were a couple of files that had a mix of states that I manually fixed up. Script isn't perfect, ran the auto fix for multiple blank lines.

Here is the script generation:

https://chatgpt.com/share/67c6065d-8738-8006-a290-28614ddb874e

Microsoft Reviewers: Open in CodeFlow

Created a simple script using ChatGPT that did most of the lifting. There were a couple of files that had a mix of states that I manually fixed up. Script isn't perfect, ran the auto fix for multiple blank lines.

Here is the script generation:

https://chatgpt.com/share/67c6065d-8738-8006-a290-28614ddb874e
@JeremyKuhne JeremyKuhne requested a review from a team as a code owner March 3, 2025 19:45
JeremyKuhne added a commit to JeremyKuhne/winforms that referenced this pull request Mar 3, 2025
Enables nullable by default in SWFDT. See dotnet#13063 for more context.
Copy link

codecov bot commented Mar 3, 2025

Codecov Report

Attention: Patch coverage is 90.32258% with 3 lines in your changes missing coverage. Please review.

Project coverage is 76.10636%. Comparing base (4bcc8a9) to head (cca5681).
Report is 9 commits behind head on main.

Additional details and impacted files
@@                 Coverage Diff                  @@
##                main      #13063          +/-   ##
====================================================
- Coverage   96.95044%   76.10636%   -20.84409%     
====================================================
  Files           1225        3275        +2050     
  Lines         357134      643803      +286669     
  Branches        5537       47445       +41908     
====================================================
+ Hits          346243      489975      +143732     
- Misses         10049      150266      +140217     
- Partials         842        3562        +2720     
Flag Coverage Δ
Debug 76.10636% <90.32258%> (-20.84409%) ⬇️
integration 18.04427% <ø> (?)
production 50.13866% <ø> (?)
test 96.95044% <90.32258%> (ø)
unit 47.54682% <ø> (?)

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

@JeremyKuhne JeremyKuhne enabled auto-merge (squash) March 4, 2025 18:16
@JeremyKuhne JeremyKuhne disabled auto-merge March 4, 2025 18:16
@JeremyKuhne JeremyKuhne enabled auto-merge (squash) March 4, 2025 18:17
@JeremyKuhne JeremyKuhne merged commit e326ed9 into dotnet:main Mar 4, 2025
8 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 10.0 Preview3 milestone Mar 4, 2025
JeremyKuhne added a commit that referenced this pull request Mar 4, 2025
Enables nullable by default in SWFDT. See #13063 for more context.
LeafShi1 pushed a commit to LeafShi1/winforms that referenced this pull request Mar 6, 2025
Created a simple script using ChatGPT that did most of the lifting. There were a couple of files that had a mix of states that I manually fixed up. Script isn't perfect, ran the auto fix for multiple blank lines.

Here is the script generation:

https://chatgpt.com/share/67c6065d-8738-8006-a290-28614ddb874e
LeafShi1 pushed a commit to LeafShi1/winforms that referenced this pull request Mar 6, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2025
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.

2 participants