Skip to content

Update entry for Willow Chargin in .mailmap and ACKS #132714

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 2 commits into from
Apr 19, 2025

Conversation

wchargin
Copy link
Contributor

No description provided.

@wchargin wchargin requested a review from a team as a code owner April 19, 2025 02:21
@wchargin
Copy link
Contributor Author

wchargin commented Apr 19, 2025

I've already updated my bpo profile. You can see my name on my website here to confirm: https://wchargin.com

I think that this should be "skip issue" and "skip news".

Prior art: #103696

@picnixz
Copy link
Member

picnixz commented Apr 19, 2025

For posterity: https://github.com/python/cpython/commits?author=wchargin and 674935b for the commit that added the original line in ACKS.

@picnixz picnixz changed the title mailmap, acks: update Willow Chargin Update entry for Willow Chargin in .mailmap and ACKS Apr 19, 2025
@picnixz picnixz enabled auto-merge (squash) April 19, 2025 14:33
@picnixz picnixz merged commit c821b71 into python:main Apr 19, 2025
39 checks passed
@wchargin wchargin deleted the wchargin-mailmap branch April 19, 2025 14:51
@wchargin
Copy link
Contributor Author

Thank you for the prompt review and merge. @picnixz, a request: I'd prefer if you didn't use my old name in your comment above. Separating myself from that name is the point of this change; I don't want to introduce new search results with the old name, or make it more likely for people who have only known me by my current name to stumble across the old one. Mentioning the original commit is totally fine with me—cross-referencing is good hygiene—but the deadnaming is unnecessary.

Thanks again.

(I tried to send this note by email, but I couldn't find contact info.)

@picnixz
Copy link
Member

picnixz commented Apr 19, 2025

Sorry for that! I've updated my comment to remove your old name.

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.

3 participants