Skip to content

Conversation

@Zombach
Copy link
Contributor

@Zombach Zombach commented Jul 30, 2024

Issue #5048
Based on this Assert.Multyple is not necessary
Based on this ShouldThrows -> ShouldThrow

Microsoft Reviewers: Open in CodeFlow

Fix NameTests => ShouldThrows -> ShouldThrow
@ghost ghost added the area-codeflow for labeling automated codeflow. intentionally a different color! label Jul 30, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jul 30, 2024
@Zombach Zombach changed the title CA1062#Aspire.Hosting.Qdrant#Path-1 CA1062#Aspire.Hosting.Redis#Path-1 Jul 30, 2024
Copy link
Member

@radical radical left a comment

Choose a reason for hiding this comment

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

LGTM 👍 Thanks for your contribution!

@radical radical merged commit 1d27b46 into dotnet:main Jul 30, 2024
@Zombach Zombach deleted the zombach/validate-arguments-of-public-methods#aspire-hosting-redis#path-1 branch August 2, 2024 16:21
@github-actions github-actions bot locked and limited conversation to collaborators Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-codeflow for labeling automated codeflow. intentionally a different color! community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants