Skip to content

Update /wholearchive command documentation to be more like other comm… #1905

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 2 commits into from
Feb 12, 2020

Conversation

zsd4yr
Copy link
Contributor

@zsd4yr zsd4yr commented Feb 11, 2020

…ands

Examples of arguments given

@PRMerger12
Copy link
Contributor

@zsd4yr : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@zsd4yr
Copy link
Contributor Author

zsd4yr commented Feb 11, 2020

We ran into an issue where the documentation was not clear enough that you could choose to specify a library or not.

@ktoliver ktoliver added the aq-pr-triaged Tracking label for the PR review team label Feb 11, 2020
Revert to standard syntax block. Add Arguments section.
@PRMerger9
Copy link
Contributor

@corob-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Copy link
Contributor

@colin-home colin-home left a comment

Choose a reason for hiding this comment

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

@zsd4yr
Approved, with changes. We adopted a new style for tool syntax when we switched to markdown, but haven't applied it everywhere yet. In the interest of addressing your confusion about the optional bracketed part in the syntax, I'm leaving it separated. Heaven help you if you need to invoke /GENPROFILE.

@colin-home colin-home merged commit 3f90d12 into MicrosoftDocs:master Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants