-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Updating authors #223
Comments
|
Just relying on the GitHub contributors page is also be fine by me. From the command line, one can run something like |
I thought we could ask people to add themselves. We do this in Zarr for example. That said, I don't feel strongly about it. Some people do like the credit though |
That was the original situation here too, but no-one ever updated that file.
Sure, some people do like credit. I'm happy to highlight the github contributors page, I think that fills this function. I'm more worried about creating only crediting some contributors & not others with a manually updated list. That feels more fraught. |
Ok, I'll make a PR to remove the Your old email address still appears in the setup.py file, not sure if you want that changed to something else @jakirkham Line 58 in 16687e5
|
Feel free to drop that one as well. Would much rather users come to GitHub than email The author field can probably be made more inclusive like |
Ok, done |
Yeah that makes sense. If people are not doing this themselves, then agree we shouldn't try to maintain the list ourselves. |
Currently this file lists only me. However other people have been contributing here. It would be great to have a more comprehensive list.
Also other projects tend to call this
CONTRIBUTORS.txt
. I thinkAUTHORS.rst
just came from a Python project template tool. So it might be worth renaming.Independently some of the source files have
__author__
and__email__
. The email referenced is dead and as already noted we have more people contributing. We might want to strip these out.Not sure what other relevant changes we might want to include, but feel free to suggest.
The text was updated successfully, but these errors were encountered: