Skip to content

Conversation

AnkitAhlawat7742
Copy link
Contributor

Issue #5050

Bug Description:
The code contains inappropriate international email test data, including references to the Ukrainian nationalist Stepan Bandera and an email address containing a nationalist slogan. This content is unsuitable and should be replaced with neutral examples to maintain professionalism and inclusivity.

@AnkitAhlawat7742
Copy link
Contributor Author

@jaraco , if possible can you please review this PR

@AnkitAhlawat7742
Copy link
Contributor Author

@sharktide , I noticed that you've approved the PR, but it hasn't been merged into the main branch yet.
If there's anything else I need to do from my side to complete the merge, please let me know.

@sharktide
Copy link

@sharktide , I noticed that you've approved the PR, but it hasn't been merged into the main branch yet.

If there's anything else I need to do from my side to complete the merge, please let me know.

I don't have write permissions to this repository. Someone else (eg. a core dev) will have to merge it

@AnkitAhlawat7742
Copy link
Contributor Author

@abravalheri , This PR has already been reviewed and approved by @sharktide, but he doesn't have the necessary permissions to merge it. Could you please go ahead and merge it, as the review is complete and the pipeline has passed?
let me know if i need to do anything else

@AnkitAhlawat7742
Copy link
Contributor Author

@jaraco, if possible, could you please merge this PR? It has been reviewed and approved by Sharktide, but they do not have the necessary permissions to merge it. And it has been pending from long time

]
maintainers = [
{name = "Степан Бандера", email = "криївка@оун-упа.укр"},
{name = "Ankit Ahlawat", email = "[email protected]"},
Copy link
Contributor

@Avasam Avasam Oct 17, 2025

Choose a reason for hiding this comment

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

Aside from any discussion about whether someone is a good name to use or not, wouldn't it be better to keep using non-ANSI chars here for testing ? I see you did so for the email below. Does this person's (your?) name also translate to (Hindi?) characters? (Google translate gives me अंकित अहलावत)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Avasam Thank you for the suggestion! I’ve updated the maintainer’s name to use Hindi characters (अंकित अहलावत) to ensure consistency with non-ASCII testing.

'Ankit Ahlawat <अंकित@उदाहरण.भारत>',
marks=pytest.mark.xfail(
reason="CPython's `email.headerregistry.Address` only supports "
'RFC 5322, as of Nov 10, 2022 and latest Python 3.11.0',
Copy link
Contributor

Choose a reason for hiding this comment

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

Worth updating the "as of" date and python version

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! That makes sense — I’ve updated the reason to reflect the current date and Python version (Oct 20, 2025 and Python 3.13.0).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Please have a look now.

@jaraco jaraco merged commit a49f6b9 into pypa:main Oct 20, 2025
6 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants