Skip to content

Repo sync for protected CLA branch#4191

Merged
colin-home merged 3 commits intomainfrom
main637998097702843921sync_temp
Sep 27, 2022
Merged

Repo sync for protected CLA branch#4191
colin-home merged 3 commits intomainfrom
main637998097702843921sync_temp

Conversation

@opbld17
Copy link
Copy Markdown
Contributor

@opbld17 opbld17 commented Sep 26, 2022

The pull request is created from main637998097702843921sync_temp to main to fix git push error for protected CLA branch

Taojunshen and others added 3 commits September 24, 2022 01:37
* Updated C6101

Mirror of my public PR. Added an example. Added proper spacing between headers and code blocks, matched formatting to my other PRs

* Updated C6101

Minor changes in response to Acrolinx report

* Updated C6101

More Acrolinx changes, split a run-on sentence

* Updated C26495

Matched formatting to my other PRs. Added more information to the example.

* Updated C26495

Fixed spelling error

* Updated C6101

Updated to the latest format

* Updated C6101

Single backticks and InOut->Inout

* Updated C26495

Updated to the new format
NOTE: This warning has no code analysis name. Per @MichaelSquires instructions, I omitted the usual keywords and 'Code analysis name:' section to match

* Update compiler-warning-level-4-c4464.md

@corob-msft 
proposal.  I don't warrant it shouldn't be worded in a better way.

* Update compiler-warning-level-4-c4464.md

typo fix

* fix: delete unnecessary asterisk

followup fix of #4136

* optimize prime test

* Update floating-point-support.md

Explain what "ulp" stands for

* Updates for cpp-docs 4161

* Update integritycheck-require-signature-check.md

* alt-text

* Update abstract-cpp-component-extensions.md

* Add syntax highlighting to atl-mfc-shared

* Updated C26495

Added code analysis ID

* alt-text updates

* customer fix

* minor updates

* Incorporate changes in rewrite

Updates for style and clarity. Also give it an Acrolinx pass.

* Clarify version support

* Tweak language for minimum support

* Add code style, comment

@MugBergerFries 
It's a good idea to use code styling for mentions of variable or class names (especially when they're easily confused generic names like "value").
I also added a comment to the fixed code sample to amplify the change and what it was doing.

* acrolinx

* cleanup pass

* [BULK] DocuTune - Rebranding

* alt-text updates

* update alt-text

* Update README.md

* Address cpp-docs 4149 __umulh on ARM64

* Add slashes per cpp-docs 4150

* update alt-text code and removed 'experimental' from images

* Update clang-support-msbuild.md

Updated for better visibility of LLVMToolsVersion property.

Note the text was also misleading because it kind of implies that LLVMToolsVersion supports side-by-side version of clang/LLVM in the same `LLVMInstallDir`, but that's not how clang/LLVM works. If you want to select a different version, you need to point it to a different path.

* test fix

* test fix

* test fix

* alt-text updates plus acrolinx

* acrolinx

* Split props file and IDE sections

When both path and version are merged in the .props file discussion, a division between the .props approach and the IDE approach seems more natural than repeating the same content twice.

* Update to add /std requirement

* Test of build checks features

* Update cpp-linter-overview.md

* Provide F1 links to articles for VS linker properties (#4529)

* Update F1 links for Linker Properties

* F1 Project: Linker Properties part 1

* Finish Linker properties F1 links

* Fix typo

* Fix acrolinx related issues

Co-authored-by: Samuel Berger <sjamesberger@gmail.com>
Co-authored-by: Brad Litterell <49081806+bradlitterell@users.noreply.github.com>
Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Colin Robertson <3836425+corob-msft@users.noreply.github.com>
Co-authored-by: opbld16 <opbld16@microsoft.com>
Co-authored-by: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com>
Co-authored-by: Courtney Wales <62625502+Court72@users.noreply.github.com>
Co-authored-by: opbld15 <opbld15@microsoft.com>
Co-authored-by: TylerMSFT <Tyler.Whitney@microsoft.com>
Co-authored-by: Bryan Gold <101299717+19BMG00@users.noreply.github.com>
Co-authored-by: Edward Breeveld <42862235+EddieBreeveld@users.noreply.github.com>
Co-authored-by: Jeff Borsecnik <36546697+jborsecnik@users.noreply.github.com>
Co-authored-by: jsuther1974 <jsuther@microsoft.com>
Co-authored-by: Austin Morton <apmorton@users.noreply.github.com>
Co-authored-by: Jak Koke <v-jokoke@microsoft.com>
Co-authored-by: HO-COOH <42881734+HO-COOH@users.noreply.github.com>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: James Barnett <v-jabarnett@microsoft.com>
Co-authored-by: Dennis Rea <v-denrea@microsoft.com>
Co-authored-by: Alex Buck <abuck@microsoft.com>
Co-authored-by: Tyler Whitney <tylerw6502@gmail.com>
Co-authored-by: Feng Xu <fenxu@microsoft.com>
Co-authored-by: Chuck Walbourn <walbourn@users.noreply.github.com>
Co-authored-by: Colin Cooper <72402153+v-ccolin@users.noreply.github.com>
* Updated C6101

Mirror of my public PR. Added an example. Added proper spacing between headers and code blocks, matched formatting to my other PRs

* Updated C6101

Minor changes in response to Acrolinx report

* Updated C6101

More Acrolinx changes, split a run-on sentence

* Updated C26495

Matched formatting to my other PRs. Added more information to the example.

* Updated C26495

Fixed spelling error

* Updated C6101

Updated to the latest format

* Updated C6101

Single backticks and InOut->Inout

* Updated C26495

Updated to the new format
NOTE: This warning has no code analysis name. Per @MichaelSquires instructions, I omitted the usual keywords and 'Code analysis name:' section to match

* Update compiler-warning-level-4-c4464.md

@corob-msft 
proposal.  I don't warrant it shouldn't be worded in a better way.

* Update compiler-warning-level-4-c4464.md

typo fix

* fix: delete unnecessary asterisk

followup fix of #4136

* optimize prime test

* Update floating-point-support.md

Explain what "ulp" stands for

* Updates for cpp-docs 4161

* Update integritycheck-require-signature-check.md

* alt-text

* Update abstract-cpp-component-extensions.md

* Add syntax highlighting to atl-mfc-shared

* Updated C26495

Added code analysis ID

* alt-text updates

* customer fix

* minor updates

* Incorporate changes in rewrite

Updates for style and clarity. Also give it an Acrolinx pass.

* Clarify version support

* Tweak language for minimum support

* Add code style, comment

@MugBergerFries 
It's a good idea to use code styling for mentions of variable or class names (especially when they're easily confused generic names like "value").
I also added a comment to the fixed code sample to amplify the change and what it was doing.

* acrolinx

* cleanup pass

* [BULK] DocuTune - Rebranding

* alt-text updates

* update alt-text

* Update README.md

* Address cpp-docs 4149 __umulh on ARM64

* Add slashes per cpp-docs 4150

* update alt-text code and removed 'experimental' from images

* Update clang-support-msbuild.md

Updated for better visibility of LLVMToolsVersion property.

Note the text was also misleading because it kind of implies that LLVMToolsVersion supports side-by-side version of clang/LLVM in the same `LLVMInstallDir`, but that's not how clang/LLVM works. If you want to select a different version, you need to point it to a different path.

* test fix

* test fix

* test fix

* alt-text updates plus acrolinx

* acrolinx

* Split props file and IDE sections

When both path and version are merged in the .props file discussion, a division between the .props approach and the IDE approach seems more natural than repeating the same content twice.

* Update to add /std requirement

* Test of build checks features

* Update cpp-linter-overview.md

* Provide F1 links to articles for VS linker properties (#4529)

* Update F1 links for Linker Properties

* F1 Project: Linker Properties part 1

* Finish Linker properties F1 links

* Fix typo

* Fix acrolinx related issues

Co-authored-by: Samuel Berger <sjamesberger@gmail.com>
Co-authored-by: Brad Litterell <49081806+bradlitterell@users.noreply.github.com>
Co-authored-by: opbld17 <opbld17@microsoft.com>
Co-authored-by: Colin Robertson <3836425+corob-msft@users.noreply.github.com>
Co-authored-by: opbld16 <opbld16@microsoft.com>
Co-authored-by: Kisaragi <48310258+KisaragiEffective@users.noreply.github.com>
Co-authored-by: Courtney Wales <62625502+Court72@users.noreply.github.com>
Co-authored-by: opbld15 <opbld15@microsoft.com>
Co-authored-by: TylerMSFT <Tyler.Whitney@microsoft.com>
Co-authored-by: Bryan Gold <101299717+19BMG00@users.noreply.github.com>
Co-authored-by: Edward Breeveld <42862235+EddieBreeveld@users.noreply.github.com>
Co-authored-by: Jeff Borsecnik <36546697+jborsecnik@users.noreply.github.com>
Co-authored-by: jsuther1974 <jsuther@microsoft.com>
Co-authored-by: Austin Morton <apmorton@users.noreply.github.com>
Co-authored-by: Jak Koke <v-jokoke@microsoft.com>
Co-authored-by: HO-COOH <42881734+HO-COOH@users.noreply.github.com>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: James Barnett <v-jabarnett@microsoft.com>
Co-authored-by: Dennis Rea <v-denrea@microsoft.com>
Co-authored-by: Alex Buck <abuck@microsoft.com>
Co-authored-by: Tyler Whitney <tylerw6502@gmail.com>
Co-authored-by: Feng Xu <fenxu@microsoft.com>
Co-authored-by: Chuck Walbourn <walbourn@users.noreply.github.com>
Co-authored-by: Colin Cooper <72402153+v-ccolin@users.noreply.github.com>
@opbld30
Copy link
Copy Markdown

opbld30 commented Sep 26, 2022

Docs Build status updates of commit 7be3cdb:

✅ 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:

@Jak-MS
Copy link
Copy Markdown
Contributor

Jak-MS commented Sep 26, 2022

#assign: @corob-msft

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged Tracking label for the PR review team label Sep 26, 2022
@colin-home colin-home merged commit 0b3e48e into main Sep 27, 2022
@colin-home colin-home deleted the main637998097702843921sync_temp branch September 27, 2022 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants