Skip to content

Restored enumeration member remarks, Part 7 #2595

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 8 commits into from
Jun 26, 2019

Conversation

rpetrusha
Copy link

Restored enumeration member remarks, Part 7

Contributes to #1306

@rpetrusha rpetrusha added this to the June 2019 milestone Jun 13, 2019
@rpetrusha rpetrusha requested a review from scottaddie as a code owner June 13, 2019 19:15
@rpetrusha rpetrusha self-assigned this Jun 13, 2019
@rpetrusha rpetrusha added the migration-fixes Indicates PRs that are related to problems caused by the migration from MSDN label Jun 13, 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.

Changes look good @rpetrusha. Left some comments for your consideration.

@@ -45,7 +45,10 @@

> [!NOTE]
Copy link
Contributor

Choose a reason for hiding this comment

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

not related to your changes, but this remarks made up mostly of notes looks weird to me

@@ -138,15 +138,7 @@
</ReturnValue>
Copy link
Contributor

@mairaw mairaw Jun 14, 2019

Choose a reason for hiding this comment

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

not related to your changes but the remarks for this type has a list of the powerful flags that should be fenced to prevent localization and that paragraph be added inside the caution note

Ron Petrusha and others added 3 commits June 18, 2019 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
migration-fixes Indicates PRs that are related to problems caused by the migration from MSDN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants