Skip to content

Repo sync for protected CLA branch #4196

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 28, 2022
Merged

Repo sync for protected CLA branch #4196

merged 2 commits into from
Sep 28, 2022

Conversation

opbld17
Copy link
Contributor

@opbld17 opbld17 commented Sep 28, 2022

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

Taojunshen and others added 2 commits September 29, 2022 01:27
@Taojunshen @MugBergerFries
@bradlitterell @opbld17 @colin-home @opbld16 @KisaragiEffective @Court72 @opbld15 @19BMG00 @EddieBreeveld @jborsecnik @jsuther1974 @apmorton @Jak-MS @HO-COOH @prmerger-automator @JamesJBarnett @denrea @alexbuckgit @ragingo @fenxu @walbourn @v-ccolin @LychyLychy @chausner @Stacyrch140 @PMEds28
9/28/2022 AM Publish (#4550)
d2b1b57 
* 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

* Fix Supported SSH algorithms

* 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

* Update dot-xmm.md

Fixed overzealous replacement

* [BULK UPDATE] DocuTune - Rebranding

* Fix missing semicolon

* Update files per cpp-docs 4181

* Add /ILK option per cpp-docs 4194

* Update per cpp-docs 4193

* Address cpp-docs 4186 code labels

* template function -> function template

* Acrolinx fixes.

* Tweak language for 4178 update

* Updates to main and wmain signatures

* Acrolinx and style updates per review

* Fix spelling, i18n, style, Acrolinx issues

* Fix date.

Co-authored-by: Samuel Berger <[email protected]>
Co-authored-by: Brad Litterell <[email protected]>
Co-authored-by: opbld17 <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: opbld16 <[email protected]>
Co-authored-by: Kisaragi <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
Co-authored-by: opbld15 <[email protected]>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: Bryan Gold <[email protected]>
Co-authored-by: Edward Breeveld <[email protected]>
Co-authored-by: Jeff Borsecnik <[email protected]>
Co-authored-by: jsuther1974 <[email protected]>
Co-authored-by: Austin Morton <[email protected]>
Co-authored-by: Jak Koke <[email protected]>
Co-authored-by: HO-COOH <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Dennis Rea <[email protected]>
Co-authored-by: Alex Buck <[email protected]>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: ragingo <[email protected]>
Co-authored-by: Feng Xu <[email protected]>
Co-authored-by: Chuck Walbourn <[email protected]>
Co-authored-by: Colin Cooper <[email protected]>
Co-authored-by: Lychy <[email protected]>
Co-authored-by: chausner <[email protected]>
Co-authored-by: Stacyrch140 <[email protected]>
Co-authored-by: Paula Miller <[email protected]>
@opbld17
Merging changes synced from https://github.com/MicrosoftDocs/cpp-docs-pr
5d1934c 
 (branch live)
@opbld17 opbld17 added the CLA label Sep 28, 2022
@prmerger-automator
Copy link
Contributor

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

@opbld33
Copy link

opbld33 commented Sep 28, 2022

Learn Build status updates of commit 5d1934c:

✅ 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-automator
Copy link
Contributor

PRMerger Results

Issue Description
Changed Files This PR contains more than 10 changed files.
File Change Percent This PR contains file(s) with more than 20% file change.

@colin-home colin-home merged commit f79ac8a into main Sep 28, 2022
@colin-home colin-home deleted the main637999830513996939sync_temp branch September 28, 2022 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@colin-home colin-home colin-home approved these changes

Assignees
No one assigned
Labels
Change sent to author CLA cpp-language/tech do-not-merge needs-human-review visual-cpp/prod
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@opbld17 @opbld33 @colin-home @Taojunshen