Skip to content

[UUF] Fix bracket indentation #10992

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
Feb 24, 2025
Merged

[UUF] Fix bracket indentation #10992

merged 3 commits into from
Feb 24, 2025

Conversation

gewarren
Copy link
Contributor

  • Fix bracket indentation
  • Modernize code

@gewarren gewarren requested a review from a team as a code owner February 22, 2025 00:26
@ghost ghost added the area-System.Runtime label Feb 22, 2025
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-runtime

This comment was marked as outdated.

Copy link

Learn Build status updates of commit 6543e72:

✅ Validation status: passed

File Status Preview URL Details
snippets/csharp/System/DateTime/ToString/tostring.csproj ✅Succeeded
snippets/csharp/System/DateTime/ToString/ToString4.cs ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

I agree with Tanner's formatting comment, and then this is ready to merge.

@gewarren gewarren enabled auto-merge (squash) February 24, 2025 16:53
Copy link

Learn Build status updates of commit a59f422:

✅ Validation status: passed

File Status Preview URL Details
snippets/csharp/System/DateTime/ToString/tostring.csproj ✅Succeeded
snippets/csharp/System/DateTime/ToString/ToString4.cs ✅Succeeded View

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren merged commit 63713eb into main Feb 24, 2025
4 checks passed
@gewarren gewarren deleted the gewarren-patch-1 branch February 24, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants