Skip to content

Add security warning to mailbox docs#149295

Open
StanFromIreland wants to merge 2 commits into
python:mainfrom
StanFromIreland:mailbox-security
Open

Add security warning to mailbox docs#149295
StanFromIreland wants to merge 2 commits into
python:mainfrom
StanFromIreland:mailbox-security

Conversation

@StanFromIreland

@StanFromIreland StanFromIreland commented May 2, 2026

Copy link
Copy Markdown
Member

CC @bitdancer, you should have gotten a series of notifications just there. I suggest adding a security warning, they collectively are the motivation for this change.

@StanFromIreland StanFromIreland requested a review from a team as a code owner May 2, 2026 17:26
@bedevere-app bedevere-app Bot added docs Documentation in the Doc dir skip news labels May 2, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs May 2, 2026
@read-the-docs-community

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

Copy link
Copy Markdown

Comment thread Doc/library/mailbox.rst Outdated
@bedevere-app

bedevere-app Bot commented May 2, 2026

Copy link
Copy Markdown

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

@StanFromIreland

Copy link
Copy Markdown
Member Author

I'll defer to RDM, closing.

@StanFromIreland StanFromIreland deleted the mailbox-security branch May 3, 2026 10:35
@StanFromIreland StanFromIreland restored the mailbox-security branch June 7, 2026 16:30
Co-authored-by: R. David Murray <rdmurray@bitdance.com>
@StanFromIreland

Copy link
Copy Markdown
Member Author

I have made the requested changes; please review again

I slightly modified your suggestion @bitdancer to smooth it out a little and to use the standard terminology.

@bedevere-app

bedevere-app Bot commented Jun 10, 2026

Copy link
Copy Markdown

Thanks for making the requested changes!

@bitdancer: please review the changes made to this pull request.

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

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

5 participants