Skip to content

Repo sync for protected CLA branch #4201

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 128 commits into from
Oct 3, 2022
Merged

Conversation

opbld16
Copy link
Contributor

@opbld16 opbld16 commented Oct 3, 2022

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

MugBergerFries and others added 30 commits July 26, 2022 13:07
Mirror of my public PR. Added an example. Added proper spacing between headers and code blocks, matched formatting to my other PRs
Minor changes in response to Acrolinx report
More Acrolinx changes, split a run-on sentence
Matched formatting to my other PRs. Added more information to the example.
Fixed spelling error
Matched formatting and wording to my other PRs. Added more information to the example
Reworded to appease high-lord Acrolinx
Matched formatting and wording to my other PRs. Matched the example to C6200 since they were functionally the same.
Fixed output
Matched formatting and wording with my other PRs. Added more information to the example
Small to changes at the behest of Acrolinx
Changed wording slightly, matched formatting and wording to my other PRs
Changed wording, acquiesced to Acrolinx's request
Changed wording to be more concise and less hostile. Changed the format and wording to match my other PRs
Acrolinx changes
Updated to the new format
Updated to the new format
Added code analysis name
Added code analysis name
Updated to the new format
Updated to the new format
Co-authored-by: Michael Squires <[email protected]>
Co-authored-by: Michael Squires <[email protected]>
Spacing fixed
Co-authored-by: Michael Squires <[email protected]>
Spacing fixed
Removed third example and corrected earlier examples. Changed 3 space indentation to 4 to be consistent with my other PRs. Fixed spacing
Consistent spacing
Updated to the latest format
Single backticks and InOut->Inout
MugBergerFries and others added 17 commits September 28, 2022 11:57
Removed extra asterisk
Incorrectly copied warning message from C6200.
Quote the actual message, bad grammar and all, to make search happy.
Simplify. Update date. Remove quotes from fully qualified, since we don't define it.
Fix date. Also tweak a sentence for readability.
* Update What's New for 17.3

* Fix link issue
* updating

* draft

* add missing iterator functions

* acrolinx and get iterator-functions.md in the build

* fixed up iterator-functions.md

* cleanup pass

* acrolinx

Co-authored-by: TylerMSFT <[email protected]>
@opbld30
Copy link

opbld30 commented Oct 3, 2022

Learn Build status updates of commit 61f6e35:

✅ 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
Added File(s) This PR contains added files. New files require human review.
Changed Files This PR contains more than 10 changed files.
Commits Count This PR contains more than 15 commits.
File Change Percent This PR contains file(s) with more than 20% file change.
Image File This PR added or updated an image file(s).

@prmerger-automator
Copy link
Contributor

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

@colin-home colin-home merged commit c415059 into main Oct 3, 2022
@colin-home colin-home deleted the main638004173927295828sync_temp branch October 3, 2022 18:16
mstorsjo pushed a commit to mstorsjo/cpp-docs that referenced this pull request Oct 4, 2022
…property-1921541-2

New article - Add an IDL Property
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.