Skip to content

Conversation

rocketsofthsv
Copy link
Contributor

@mairaw mairaw added the vendor-project Indicates the issue/pr is related to a vendor project. label Dec 3, 2019
@mairaw mairaw self-requested a review December 4, 2019 00:41
@mairaw mairaw added this to the December 2019 milestone Dec 4, 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 @rocketsofthsv. Left a few comments for you to consider before we can merge this.

@mairaw
Copy link
Contributor

mairaw commented Dec 4, 2019

You can batch all my suggestions into a single commit via the Files changes tab (https://github.com/dotnet/dotnet-api-docs/pull/3573/files).

@rocketsofthsv
Copy link
Contributor Author

@mairaw committed your suggestions

@mairaw
Copy link
Contributor

mairaw commented Dec 4, 2019

I still see some not commited. Go to https://github.com/dotnet/dotnet-api-docs/pull/3573/files to see them all and add them in batch.

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.

Thank you @rocketsofthsv!

@mairaw mairaw merged commit 0e9bccd into dotnet:master Dec 4, 2019
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