Skip to content

Conversation

WilliamAntonRohm
Copy link
Contributor

@WilliamAntonRohm WilliamAntonRohm commented Oct 22, 2019

samples PR already merged -- dotnet/samples#1654

@mairaw mairaw added the vendor-project Indicates the issue/pr is related to a vendor project. label Oct 22, 2019
@mairaw mairaw added this to the October 2019 milestone Oct 22, 2019
Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

Thanks @WilliamAntonRohm. Left a couple of comments. Can you also make sure your other PRs don't have the same issue?

@WilliamAntonRohm
Copy link
Contributor Author

@mairaw -- Maira, this one is ready for your review.

@mairaw mairaw closed this Dec 4, 2019
@mairaw mairaw reopened this Dec 4, 2019
@mairaw
Copy link
Contributor

mairaw commented Dec 4, 2019

Need a new build

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

A few more things to be addressed before we can merge

WilliamAntonRohm and others added 6 commits December 8, 2019 14:29
Co-Authored-By: Maira Wenzel <[email protected]>
Co-Authored-By: Maira Wenzel <[email protected]>
Co-Authored-By: Maira Wenzel <[email protected]>
Co-Authored-By: Maira Wenzel <[email protected]>
Co-Authored-By: Maira Wenzel <[email protected]>
Co-Authored-By: Maira Wenzel <[email protected]>
@WilliamAntonRohm
Copy link
Contributor Author

@mairaw -- thank you for your detailed review. I believe this PR is good to go!

Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

Left a few additional comments to be addressed

Copy link
Contributor Author

@WilliamAntonRohm WilliamAntonRohm left a comment

Choose a reason for hiding this comment

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

(again, I'm not sure how or why GitHub started a review for me)

@WilliamAntonRohm
Copy link
Contributor Author

@mairaw -- Maira, please review these changes if this build ever finishes :)

@mairaw mairaw modified the milestones: October 2019, January 2020 Jan 14, 2020
Copy link
Contributor

@mairaw mairaw left a comment

Choose a reason for hiding this comment

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

@mairaw mairaw merged commit e272112 into dotnet:master Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
vendor-project Indicates the issue/pr is related to a vendor project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants