Skip to content

CoreCLR: Automatically port System.GC* triple slash comments to Docs #2759

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jul 17, 2019

Conversation

carlossanlop
Copy link
Contributor

@carlossanlop carlossanlop commented Jul 15, 2019

Found some triple slash comments in Garbage Collection related APIs that could be automatically ported.

issue-guide.md is not clear about the ownership of these APIs, so I'm going to ask the people who have modified these APIs most recently to take a look or help me find the right owner:

@luhenry @VSadov @stephentoub @jkotas @Andy-MS

@rpetrusha @mairaw for language review.

@carlossanlop carlossanlop self-assigned this Jul 15, 2019
@carlossanlop carlossanlop added 🏁 Release: .NET Core 3.0 :checkered_flag: Release: .NET Core 3.0 new-content Indicates PRs that contain new articles waiting-on-reviews Indicates PRs that cannot be merged because of the lack of reviews labels Jul 15, 2019
Copy link
Member

@VSadov VSadov left a comment

Choose a reason for hiding this comment

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

Can't comment on style, but technically the comments look correct.

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.

I've made a number of changes, @carlossanlop. Please review them before I merge.

@rpetrusha rpetrusha added verify-build-before-merge and removed waiting-on-reviews Indicates PRs that cannot be merged because of the lack of reviews labels Jul 16, 2019
@carlossanlop
Copy link
Contributor Author

@rpetrusha thanks for your changes, they look good to me. I think this is ready to merge.

@rpetrusha
Copy link

Thanks, @carlossanlop. I'll merge now.

@rpetrusha rpetrusha merged commit c2d3b35 into dotnet:master Jul 17, 2019
@carlossanlop carlossanlop deleted the clr_System.GC branch July 18, 2019 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏁 Release: .NET Core 3.0 :checkered_flag: Release: .NET Core 3.0 new-content Indicates PRs that contain new articles
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants