Skip to content

Conversation

@crummel
Copy link
Contributor

@crummel crummel commented Mar 17, 2023

@ghost ghost added the area-additional-repos Adding additional contributing repos label Mar 17, 2023
@MichaelSimons
Copy link
Member

@crummel - It appears that the dry-run you linked to failed prior to invoking the affected code. Could you please link to a PR created by these changes?

@crummel
Copy link
Contributor Author

crummel commented Mar 20, 2023

Copy link
Member

@MichaelSimons MichaelSimons left a comment

Choose a reason for hiding this comment

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

One minor suggestion/question. The rest looks good.

Co-authored-by: Michael Simons <msimons@microsoft.com>
@crummel crummel enabled auto-merge (squash) March 20, 2023 19:41
@crummel crummel requested a review from premun March 20, 2023 19:42
Copy link
Member

@MichaelSimons MichaelSimons left a comment

Choose a reason for hiding this comment

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

Friendly reminder to cleanup the internal PR as part of merging.

@crummel crummel merged commit 90457ea into dotnet:main Mar 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-additional-repos Adding additional contributing repos

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatically submit update PRs to VMR-lite

2 participants