discretize icon indicating copy to clipboard operation
discretize copied to clipboard

Published 20 hours ago • verified publisher icon simpeg

AUTHORS.rst

Open lheagy opened this issue 7 years ago • 4 comments

The author list is out of date. Is this something we want to be maintaining manually? or should we just point to github contributions?

cc @rowanc1

lheagy avatar Dec 16 '17 23:12 lheagy

I think keeping this manually updated is good? I quite like what @leouieda has done in fatiando with a PR checklist.

rowanc1 avatar Dec 17 '17 20:12 rowanc1

The checklist is good for new users but it can become a bit of a pain for people making lots of PRs. We sometimes end up just ignoring it, which is not good. Not sure what the best way forward would be. Ideally, a bot that comments on the PR reminding the author but that is a lot of work.

leouieda avatar Dec 17 '17 21:12 leouieda

@lheagy @rowanc1 coming back to this, there is a Github bot called "Welcome Bot" that will post whatever you tell it to when a new user submits a PR, has one merged, or opens a new issue. I've been using it to tell people to submit a new PR adding themselves to the authors file (see the config file). This way the PRs won't have merge conflicts all the time.

What do you think? I could open a PR adding the configurations but I think someone with admin rights needs to enable the app.

leouieda avatar Oct 24 '18 02:10 leouieda

Hi @leouieda: thanks for resurrecting this. I think this would be a very positive addition! We have started sketching out a contributors guide (mostly based off of the fatiando ecosystem 😄) in simpeg/community#3. I think we could get a draft of that in shortly so that there are live links to point to.

Feel free to start a PR and I can enable the app

lheagy avatar Oct 24 '18 02:10 lheagy