Skip to content
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

Add contributors to package.json #287

Closed
haadcode opened this issue Dec 23, 2017 · 9 comments
Closed

Add contributors to package.json #287

haadcode opened this issue Dec 23, 2017 · 9 comments

Comments

@haadcode
Copy link
Member

haadcode commented Dec 23, 2017

Hey everyone,

I wanted to gauge your interest in getting some acknowledgment for your contributions on OrbitDB. The js-ipfs project includes everyone who ever contributed in their package.json.

This gives recognition for the contributions made by people over time, which I like a lot. The only down sides I can see are: it requires making sure the list of contributors is up to date (not a blocker to do it imo) and as everyone's emails are included, some people may not appreciate having their email "published" and the fact that we've had contributors "outside" repo contributions (issues, IRC discussions, etc) that I feel should be credited just the same for the value they've added to the project (and this is hard to make sure that everyone's included).

I would be more than happy to start listing everyone in the package.json. I wanted to ask how do you feel about it? Or if you have alternative places where we can give credit to people?

cc @dignifiedquire @prabodhmeshram @ricardojmendez @shamb0t @RichardLitt @Alex-Werner @Kubuxu @nightwolfz @atfornes (I'm sorry if I missed anyone)

@dignifiedquire
Copy link
Contributor

dignifiedquire commented Dec 23, 2017 via email

@ricardojmendez
Copy link
Contributor

That's a nice touch, but not a deal-breaker for me if it doesn't happen. :-)

I've often seen it done on the README or CHANGELOG, but that mostly works for sporadic contributions. Adding people to the package.json sounds like a good approach.

@RichardLitt
Copy link
Member

RichardLitt commented Dec 29, 2017

Adding to the package.json makes the most sense, to me. You can also add them to the README, or to the CONTRIBUTORS file. all-contributors is an interesting spec for doing this.

You can use name-your-contributors to get all of your contributors, including discussions, which solves @dignifiedquire's issue of only getting code contributions.

Updating using a post-commit script should be doable, too, so that it stays up to date.

If you do decide on a solution - you control here, and everyone who contributes should opt-out of these lists to make it easier for you (as opposed to opt-in) - let me know if you'd like help implementing it.

@ricardojmendez
Copy link
Contributor

If you do decide on a solution - you control here, and everyone who contributes should opt-out of these lists to make it easier for you (as opposed to opt-in)

Sounds like a good idea. Could be specified on the Contribution Guidelines, that way people are aware.

@haadcode
Copy link
Member Author

I think I personally like CONTRIBUTORS file the most and @RichardLitt's proposal. What do you all think?

I don't see why we shouldn't go ahead and do this. Anyone willing to take this and implement it?

@haadcode
Copy link
Member Author

Could be specified on the Contribution Guidelines, that way people are aware.

We should add Contribution Guidelines too.

@RichardLitt
Copy link
Member

I got this.

@RichardLitt
Copy link
Member

By running:

$ name-your-contributors -u orbitdb -r orbit-db > contributors.json
$ cat contributors.json | jq '[ .[][].login | " - [@ \(.)](https://github.com/\(.))"]' | jq unique | jq -r .[] |

I was able to come up with a list of all contributors in this format:

 - [@ 5310](https://github.com/5310)
  1. Is this a good format? Perhaps First Last <github url> is better
  2. Should we be gathering emails, too? (I don't think so.)
  3. I'm not sure how to set this up with CircleCI, but it should work fairly well with Travis. @haadcode do you know the manifest well enough to run that command above?

@RichardLitt
Copy link
Member

This has now been implemented. Thank you, everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants