Skip to content

Repo sync for protected CLA branch #3819

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 3 commits into from
Apr 8, 2022
Merged

Repo sync for protected CLA branch #3819

merged 3 commits into from
Apr 8, 2022

Conversation

opbld16
Copy link
Contributor

@opbld16 opbld16 commented Apr 8, 2022

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

Taojunshen and others added 3 commits April 8, 2022 01:24
* Broken link fixed

* Broken link fixed

* Update virtual-functions.md

* Address cpp-docs 3780 3782 3790 3791 3805

* Attempt correction of table format

Co-authored-by: atikmapari <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: PRMerger19 <[email protected]>
Co-authored-by: Ming Ho <[email protected]>
Co-authored-by: PRMerger5 <[email protected]>
Co-authored-by: Regan Downer <[email protected]>
* Update open-wopen.md

Clarify that _O_TEMPORARY sets FILE_SHARE_DELETE.

* Update other pages that mention _O_TEMPORARY effects.

* Broken link fixed

* Broken link fixed

* Update virtual-functions.md

* Address cpp-docs 3780 3782 3790 3791 3805

* Attempt correction of table format

* Add documentation for compiler error C2956 and update error text (#4176)

* Add documentation for compiler error C2956 and update error text

* Add TOC entry

* Update for grammar and my own comprehension.

Sometimes, you just get carried away. This version could use some simplification.

* Simplify.

Address remarks, attempt simplification.
Acrolinx all the things.

* More updates per review

Co-authored-by: Colin Robertson <[email protected]>

* Address cpp-docs 3816

* Address non-blocking issues from 4207

* fix for sample program.

Co-authored-by: Steve Wishnousky <[email protected]>
Co-authored-by: atikmapari <[email protected]>
Co-authored-by: Colin Robertson <[email protected]>
Co-authored-by: PRMerger19 <[email protected]>
Co-authored-by: Ming Ho <[email protected]>
Co-authored-by: PRMerger5 <[email protected]>
Co-authored-by: Regan Downer <[email protected]>
Co-authored-by: Jonathan Emmett <[email protected]>
Co-authored-by: Tamara K <[email protected]>
Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: PRMerger17 <[email protected]>
@PRMerger15
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 9ca812d into main Apr 8, 2022
@colin-home colin-home deleted the main637850370761840685 branch April 8, 2022 18:27
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