RIGHT-TO-LEFT OVERRIDE in AUTHORS.txt

[hroncok]

Description

Hello,

in #12046 I have removed RIGHT-TO-LEFT OVERRIDE from AUTHORS.txt

It was added back in ae23f96

I suspect the file is updated by a script. I am opening this issue to track a modification of that script, so it is not added again.

Expected behavior

No response

pip version

23.2+

Python version

any

OS

any

How to Reproduce

Examine the file on GitHub. It tells you:

This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below.

Output

No response

Code of Conduct

• I agree to follow the PSF Code of Conduct.

[pfmoore]

That file is created via the code in https://github.com/pypa/pip/blob/main/tools/release/__init__.py#L48

It looks like it gets the data from git author information, by running git log. I don't have a good idea as to how we can avoid this, TBH.

Does the RIGHT-TO-LEFT OVERRIDE chacater actually cause an issue?

[hroncok]

I can probably add some text processing to that function. Feel free to assign this issue to me.

Does the RIGHT-TO-LEFT OVERRIDE chacater actually cause an issue?

I don't think so. It's being flagged by a CentOS Steram static RPM analysis tool. If you prefer me not to change it, let me know and I'll keep applying a downstream-only change to "satisfy the linter". However, since my original PR was accepted, I assumed changing the automation would be acceptable as well.

[uranusjr]

A right-to-left override character at the end of the name doesn’t have a practical effect so maybe we could just strip that. I would be fine if a PR is raised to add this auto removal logic to the script.

[hroncok]

See #12517

Took me a while to get to it, sorry about that.