Skip to content

Adding public API test coverage for Aspire.Hosting.MongoDB#5201

Merged
sebastienros merged 6 commits intomicrosoft:mainfrom
Alirexaa:validate-arguments-of-public-methods#aspire-hosting-mongodb
Aug 14, 2024
Merged

Adding public API test coverage for Aspire.Hosting.MongoDB#5201
sebastienros merged 6 commits intomicrosoft:mainfrom
Alirexaa:validate-arguments-of-public-methods#aspire-hosting-mongodb

Conversation

@Alirexaa
Copy link
Copy Markdown
Contributor

@Alirexaa Alirexaa commented Aug 6, 2024

Contributes to #2649 and #5047

Microsoft Reviewers: Open in CodeFlow

@ghost ghost added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Aug 6, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Aug 6, 2024
@mitchdenny
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
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.

Other than the few comments, LGTM.

Alirexaa and others added 2 commits August 7, 2024 11:38
Co-authored-by: Ankit Jain <radical@gmail.com>
@sebastienros sebastienros enabled auto-merge (squash) August 9, 2024 17:45
@sebastienros sebastienros merged commit 6b295c5 into microsoft:main Aug 14, 2024
@Alirexaa Alirexaa deleted the validate-arguments-of-public-methods#aspire-hosting-mongodb branch August 14, 2024 07:32
@github-actions github-actions bot locked and limited conversation to collaborators Sep 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication 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.

4 participants