Skip to content

Conversation

carlossanlop
Copy link
Contributor

No description provided.

@carlossanlop carlossanlop added new-content Indicates PRs that contain new articles waiting-on-reviews Indicates PRs that cannot be merged because of the lack of reviews 🏁 Release: .NET Core 2.x Identifies work items for the .NET Core 2.x releases labels Sep 16, 2019
@carlossanlop carlossanlop added this to the September 2019 milestone Sep 16, 2019
@carlossanlop carlossanlop self-assigned this Sep 16, 2019
Copy link

@rpetrusha rpetrusha left a comment

Choose a reason for hiding this comment

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

This file didn't actually have any documentation, @carlossanlop. I've added a type description and documentation for its two members. Please review them, since they probably leave something to be desired.

@carlossanlop
Copy link
Contributor Author

Thanks for adding the rest of the missing documentation, @rpetrusha.

Copy link
Contributor

@sdmaclea sdmaclea left a comment

Choose a reason for hiding this comment

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

LGTM

@rpetrusha rpetrusha merged commit 430c85a into dotnet:master Oct 11, 2019
@rpetrusha rpetrusha removed the waiting-on-reviews Indicates PRs that cannot be merged because of the lack of reviews label Oct 11, 2019
@carlossanlop carlossanlop deleted the ResourceSectionBuilder branch November 6, 2019 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏁 Release: .NET Core 2.x Identifies work items for the .NET Core 2.x releases new-content Indicates PRs that contain new articles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants