jaraco / inflect

Correctly generate plurals, ordinals, indefinite articles; convert numbers to words
https://pypi.org/project/inflect
MIT License
971 stars 106 forks source link

chmod +x #141

Closed DimitriPapadopoulos closed 2 years ago

DimitriPapadopoulos commented 3 years ago

The setup.py file is supposed to be executable, mark it as executable in Git.

jaraco commented 2 years ago

Thanks for the contrib, but I'll decline. setup.py doesn't need to be executable and in fact it should almost never be executed.

DimitriPapadopoulos commented 2 years ago

Sure, but then why does it need the initial #!/usr/bin/env python?

jaraco commented 2 years ago

I traced that behavior back to https://github.com/jaraco/jaraco.develop/commit/96a7ef77d7cb80a4c0539fb14daf2ea16d4499d2 . It was a bad idea. I could fix it in jaraco.skeleton, but the file is going away anyway, so no need to fuss about the shebang.