Skip to content

gh-141623: operator.rst: don't use bitwise keyword#141846

Open
doronbehar wants to merge 1 commit into
python:mainfrom
doronbehar:patch-1
Open

gh-141623: operator.rst: don't use bitwise keyword#141846
doronbehar wants to merge 1 commit into
python:mainfrom
doronbehar:patch-1

Conversation

@doronbehar
Copy link
Copy Markdown

@doronbehar doronbehar commented Nov 22, 2025

IMO this change should be backported to the stable release branches.


📚 Documentation preview 📚: https://cpython-previews--141846.org.readthedocs.build/

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 2, 2026

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label May 2, 2026
@doronbehar
Copy link
Copy Markdown
Author

doronbehar commented May 2, 2026 via email

@github-actions github-actions Bot removed the stale Stale PR or inactive for long period of time. label May 16, 2026
Comment thread Doc/library/operator.rst Outdated
Comment thread Doc/library/operator.rst Outdated
@bedevere-app
Copy link
Copy Markdown

bedevere-app Bot commented May 30, 2026

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels May 30, 2026
@read-the-docs-community
Copy link
Copy Markdown

read-the-docs-community Bot commented May 30, 2026

Comment thread Doc/library/operator.rst Outdated
+-----------------------+-------------------------+---------------------------------------+
| Bitwise Exclusive Or | ``a ^ b`` | ``xor(a, b)`` |
| Bitwise Exclusive Or, | ``a ^ b`` | ``xor(a, b)`` |
| or, Symmetric | | |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think a comma before "Symmetric" is not needed.

I am not sure that commas before "or" are needed, but I am bad in English. cc @StanFromIreland Maybe remove commas before "or" or remove "or" and keep commas?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Fixed the extra comma issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting changes docs Documentation in the Doc dir needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes skip news

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

5 participants