Skip to content

Make built-in operators title consistent #4963

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 1 commit into from
Mar 1, 2024
Merged

Make built-in operators title consistent #4963

merged 1 commit into from
Mar 1, 2024

Conversation

Rageking8
Copy link
Contributor

Continuation of #4960 for all built-in operators, together with a few cleanups and minor improvements.

@prmerger-automator prmerger-automator bot requested a review from TylerMSFT March 1, 2024 10:23
@prmerger-automator PRMerger Automator
Copy link
Contributor

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

@learn-build-service-prod Learn Build Service (PROD)
Copy link
Contributor

Learn Build status updates of commit 8d4d154:

✅ Validation status: passed

File Status Preview URL Details
docs/cpp/additive-operators-plus-and.md ✅Succeeded
docs/cpp/assignment-operators.md ✅Succeeded
docs/cpp/bitwise-exclusive-or-operator-hat.md ✅Succeeded
docs/cpp/bitwise-inclusive-or-operator-pipe.md ✅Succeeded
docs/cpp/comma-operator.md ✅Succeeded
docs/cpp/conditional-operator-q.md ✅Succeeded
docs/cpp/equality-operators-equal-equal-and-exclpt-equal.md ✅Succeeded
docs/cpp/explicit-type-conversion-operator-parens.md ✅Succeeded
docs/cpp/function-call-operator-parens.md ✅Succeeded
docs/cpp/indirection-operator-star.md ✅Succeeded
docs/cpp/logical-negation-operator-exclpt.md ✅Succeeded
docs/cpp/logical-or-operator-pipe-pipe.md ✅Succeeded
docs/cpp/multiplicative-operators-and-the-modulus-operator.md ✅Succeeded
docs/cpp/one-s-complement-operator-tilde.md ✅Succeeded
docs/cpp/postfix-increment-and-decrement-operators-increment-and-decrement.md ✅Succeeded
docs/cpp/prefix-increment-and-decrement-operators-increment-and-decrement.md ✅Succeeded
docs/cpp/relational-operators-equal-and-equal.md ✅Succeeded
docs/cpp/scope-resolution-operator.md ✅Succeeded
docs/cpp/sizeof-operator.md ✅Succeeded
docs/cpp/subscript-operator.md ✅Succeeded
docs/cpp/typeid-operator.md ✅Succeeded
docs/cpp/unary-plus-and-negation-operators-plus-and.md ✅Succeeded
docs/cpp/uuidof-operator.md ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@prmerger-automator PRMerger Automator
Copy link
Contributor

PRMerger Results

Issue Description
Changed Files This PR contains more than 10 changed files.

@Jak-MS
Copy link
Contributor

Jak-MS commented Mar 1, 2024

@TylerMSFT

  • Can you review this PR?
  • IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#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 Mar 1, 2024
TylerMSFT
TylerMSFT approved these changes Mar 1, 2024
View reviewed changes
Copy link
Collaborator

@TylerMSFT TylerMSFT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Love it - thank you.

@TylerMSFT
Copy link
Collaborator

#sign-off

@prmerger-automator PRMerger Automator
Copy link
Contributor

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

@prmerger-automator prmerger-automator bot requested a review from TylerMSFT March 1, 2024 17:50
@TylerMSFT TylerMSFT merged commit 9271243 into MicrosoftDocs:main Mar 1, 2024
learn-build-service-prod bot added a commit that referenced this pull request Mar 2, 2024
@Taojunshen @prmerger-automator
@learn-build-service-prod @mspll @sinemakinci1 @PhilKang0704 @v-alje @TylerMSFT @Court72 @Jak-MS @stoopdapoop @TartanLlama @JamesJBarnett @Rageking8
3/1/2024 Publish (#5223)
7ee48de 
* Fixes #3014

* Confirm merge from FromPublicMasterBranch to main to sync with https://github.com/MicrosoftDocs/cpp-docs (branch main) (#5198)

* largeaddressaware:no is not recommended for 64-bit apps

* Update connect-to-your-remote-linux-computer.md

Removing references to RSA/DSA keys due to security concerns

* Update largeaddressaware-handle-large-addresses.md

* Update largeaddressaware-handle-large-addresses.md

* Update largeaddressaware-handle-large-addresses.md

* Fixed syntax error in example message

* Update clang-tidy.md

Adding support for CMake Presets

* Update clang-tidy.md

addressing some feedback

* Update assignment-operators.md

Clarify built-in behaviour for compound assignment.
Add mention of overloaded compound assignment operators
Fixes #4745

* Update gmtime-s-gmtime32-s-gmtime64-s.md

Remove note about daylight savings time, as these functions convert from time-since-epoch to UTC, neither of which are affected by daylight savings

* Update gmtime-gmtime32-gmtime64.md

* Correct JSON output of sourcedependencies.md

Updates the version number in the JSON to the one currently used by the compiler, and removes the bogus ImportedHeaderUnit entry

* update date

* Update gmtime-gmtime32-gmtime64.md

update ms.date

* Update gmtime-s-gmtime32-s-gmtime64-s.md

update date

* Update assignment-operators.md

update ms.date

* Repo sync for protected branch (#4958)

* add link to backend improvements

* change feedback system from github issues to the new UUF system (#5208)

Co-authored-by: TylerMSFT <[email protected]>

* new article: include diagnostics
* acrolinx and edit
* acrolinx
* small edits
* clarified specific to C++
* clarify that this is C++ include diagnostics
* fix github #4953

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: Phil <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>

* Update clang-tidy.md

* Update clang-tidy.md

* Update clang-tidy.md

* header casing & markdown syntax tweaks

* Update docs/code-quality/clang-tidy.md

* Fix built-in operators toc yml

* Update subscript-operator.md

* Update left-shift-and-right-shift-operators-input-and-output.md

* Update subscript-operator.md

* Improve C2548 (#4964)

* Improve C2548

* Update compiler-error-c2548.md

* Update compiler-error-c2548.md

fixed title casing

* Update compiler-error-c2548.md

typo

---------

Co-authored-by: Tyler Whitney <[email protected]>

* Remove extra trailing comma in istream iterator (#4962)

* Make built-in operators title consistent (#4963)

* Fixes #3014 (#5221)

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

---------

Co-authored-by: mspll <[email protected]>
Co-authored-by: Sinem Akinci <[email protected]>
Co-authored-by: Phil <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: Alma Jenks <[email protected]>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
Co-authored-by: Jak Koke <[email protected]>
Co-authored-by: Javad Kouchakzadeh <[email protected]>
Co-authored-by: Sy Brand <[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: James Barnett <[email protected]>
Co-authored-by: Rageking8 <[email protected]>
Co-authored-by: Taojunshen <[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: mspll <[email protected]>
Co-authored-by: Sinem Akinci <[email protected]>
Co-authored-by: Phil <[email protected]>
Co-authored-by: Alma Jenks <[email protected]>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
Co-authored-by: Jak Koke <[email protected]>
Co-authored-by: Javad Kouchakzadeh <[email protected]>
Co-authored-by: Sy Brand <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Rageking8 <[email protected]>
@Rageking8 Rageking8 deleted the make-built-in-operators-title-consistent branch March 3, 2024 10:29
learn-build-service-prod bot added a commit that referenced this pull request Mar 4, 2024
@learn-build-service-prod @mspll
@sinemakinci1 @PhilKang0704 @v-alje @TylerMSFT @Court72 @Jak-MS @stoopdapoop @TartanLlama @prmerger-automator @JamesJBarnett @Rageking8 @Taojunshen
Confirm merge from FromPublicMasterBranch to main to sync with https:…
8445740 
…//github.com/MicrosoftDocs/cpp-docs (branch main) (#5198)

* largeaddressaware:no is not recommended for 64-bit apps

* Update connect-to-your-remote-linux-computer.md

Removing references to RSA/DSA keys due to security concerns

* Update largeaddressaware-handle-large-addresses.md

* Update largeaddressaware-handle-large-addresses.md

* Update largeaddressaware-handle-large-addresses.md

* Fixed syntax error in example message

* Update clang-tidy.md

Adding support for CMake Presets

* Update clang-tidy.md

addressing some feedback

* Update assignment-operators.md

Clarify built-in behaviour for compound assignment.
Add mention of overloaded compound assignment operators
Fixes #4745

* Update gmtime-s-gmtime32-s-gmtime64-s.md

Remove note about daylight savings time, as these functions convert from time-since-epoch to UTC, neither of which are affected by daylight savings

* Update gmtime-gmtime32-gmtime64.md

* Correct JSON output of sourcedependencies.md

Updates the version number in the JSON to the one currently used by the compiler, and removes the bogus ImportedHeaderUnit entry

* update date

* Update gmtime-gmtime32-gmtime64.md

update ms.date

* Update gmtime-s-gmtime32-s-gmtime64-s.md

update date

* Update assignment-operators.md

update ms.date

* Repo sync for protected branch (#4958)

* add link to backend improvements

* change feedback system from github issues to the new UUF system (#5208)

Co-authored-by: TylerMSFT <[email protected]>

* new article: include diagnostics
* acrolinx and edit
* acrolinx
* small edits
* clarified specific to C++
* clarify that this is C++ include diagnostics
* fix github #4953

---------

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: Phil <[email protected]>
Co-authored-by: James Barnett <[email protected]>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>

* Update clang-tidy.md

* Update clang-tidy.md

* Update clang-tidy.md

* header casing & markdown syntax tweaks

* Update docs/code-quality/clang-tidy.md

* Fix built-in operators toc yml

* Update subscript-operator.md

* Update left-shift-and-right-shift-operators-input-and-output.md

* Update subscript-operator.md

* Improve C2548 (#4964)

* Improve C2548

* Update compiler-error-c2548.md

* Update compiler-error-c2548.md

fixed title casing

* Update compiler-error-c2548.md

typo

---------

Co-authored-by: Tyler Whitney <[email protected]>

* Remove extra trailing comma in istream iterator (#4962)

* Make built-in operators title consistent (#4963)

* Fixes #3014 (#5221)

Co-authored-by: TylerMSFT <[email protected]>
Co-authored-by: prmerger-automator[bot] <40007230+prmerger-automator[bot]@users.noreply.github.com>

---------

Co-authored-by: mspll <[email protected]>
Co-authored-by: Sinem Akinci <[email protected]>
Co-authored-by: Phil <[email protected]>
Co-authored-by: learn-build-service-prod[bot] <113403604+learn-build-service-prod[bot]@users.noreply.github.com>
Co-authored-by: Alma Jenks <[email protected]>
Co-authored-by: Learn Build Service GitHub App <Learn Build Service [email protected]>
Co-authored-by: Tyler Whitney <[email protected]>
Co-authored-by: Courtney Wales <[email protected]>
Co-authored-by: Jak Koke <[email protected]>
Co-authored-by: Javad Kouchakzadeh <[email protected]>
Co-authored-by: Sy Brand <[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: James Barnett <[email protected]>
Co-authored-by: Rageking8 <[email protected]>
Co-authored-by: Taojunshen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TylerMSFT TylerMSFT Awaiting requested review from TylerMSFT

Assignees

@TylerMSFT TylerMSFT

Labels
aq-pr-triaged Tracking label for the PR review team Change sent to author cpp-lang/subsvc needs-human-review ready-to-merge visual-cpp/svc
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Rageking8 @Jak-MS @TylerMSFT