-
Notifications
You must be signed in to change notification settings - Fork 967
Update vcperf-commands.md #4375
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
Update vcperf-commands.md #4375
Conversation
Update commands according to https://github.com/microsoft/vcperf#command-line-reference
@NelsonDaniel : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
@NelsonDaniel : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 491bc61: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
Learn Build status updates of commit 1312c8d: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
#label:"aq-pr-triaged" |
@@ -18,23 +18,23 @@ This article lists and describes the commands available in *`vcperf.exe`*, and h | |||
## Commands to start and stop traces | |||
|
|||
> [!IMPORTANT] | |||
> The following commands all require administrative privileges. | |||
> Unless /noadmin is provided, the following commands require administrative privileges. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider adding this note "Unless /noadmin is provided" in the vcperf documentation on GitHub as well.
@Jak-MS I reviewed the PR. I will let @NelsonDaniel sign off when he is ready. |
Hi @kevcadieux Thanks for reviewing the PR! FYI - in the public repos, PRMerger will accept sign-off from the listed author only (hence the need for you to add a Thanks. |
Which version of the product do the /noadmin, /templates, and /levelN options appear in? Stashing this update for Acrolinx and i18n here:
|
Here is the release version: https://github.com/microsoft/vcperf/releases/tag/2.1.22062801 |
@NelsonDaniel : Thanks for your contribution! The author(s) have been notified to review your proposed change. |
Learn Build status updates of commit 8fb952a: ✅ Validation status: passed
For more details, please refer to the build report. Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report. For any questions, please:
|
PRMerger Results
|
@NelsonDaniel Thanks for the update. I assume since @kevcadieux made only one comment that it otherwise passed his review, so I merged it. |
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#4817) * Update vcperf-commands.md (#4375) * Update vcperf-commands.md Update commands according to https://github.com/microsoft/vcperf#command-line-reference * Update vcperf-commands.md * Update with grammar fixes for i18n --------- Co-authored-by: Colin Robertson <[email protected]> * Update tutorial-import-stl-named-module.md (#4429) * Update tutorial-import-stl-named-module.md The Title of the document doesn't reveal that this procedure is only for a command line case. Suggest altering the titles to reflect this and improve clarity for readers searching for module documentation. * Update tutorial-import-stl-named-module.md Changed the capitalization at Microsoft's request. --------- Co-authored-by: huypub <[email protected]> Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com> Co-authored-by: Nelson Daniel Troncoso <[email protected]> Co-authored-by: Colin Robertson <[email protected]> Co-authored-by: Centurion Maximus <[email protected]>
* rewording ReadString documentation to be more accurate (#4815) * reword nmax parameter wording for ReadString to be more accurate * change suggested directory location * acrolinx * incorp review * fix path * improve path * Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#4817) * Update vcperf-commands.md (#4375) * Update vcperf-commands.md Update commands according to https://github.com/microsoft/vcperf#command-line-reference * Update vcperf-commands.md * Update with grammar fixes for i18n --------- Co-authored-by: Colin Robertson <[email protected]> * Update tutorial-import-stl-named-module.md (#4429) * Update tutorial-import-stl-named-module.md The Title of the document doesn't reveal that this procedure is only for a command line case. Suggest altering the titles to reflect this and improve clarity for readers searching for module documentation. * Update tutorial-import-stl-named-module.md Changed the capitalization at Microsoft's request. --------- Co-authored-by: huypub <[email protected]> Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com> Co-authored-by: Nelson Daniel Troncoso <[email protected]> Co-authored-by: Colin Robertson <[email protected]> Co-authored-by: Centurion Maximus <[email protected]> * fix casing and wording * acrolinx --------- Co-authored-by: Anju del Moral Gonzalez <[email protected]> Co-authored-by: TylerMSFT <[email protected]> Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com> Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com> Co-authored-by: huypub <[email protected]> Co-authored-by: Nelson Daniel Troncoso <[email protected]> Co-authored-by: Colin Robertson <[email protected]> Co-authored-by: Centurion Maximus <[email protected]> Co-authored-by: Jak Koke <[email protected]>
Update commands documentation according to https://github.com/microsoft/vcperf#command-line-reference