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

Recognizing Contributions BIG and small #981

Closed
1 of 4 tasks
virkt25 opened this issue Feb 9, 2018 · 12 comments
Closed
1 of 4 tasks

Recognizing Contributions BIG and small #981

virkt25 opened this issue Feb 9, 2018 · 12 comments
Assignees
Labels
Internal Tooling Issues related to our tooling and monorepo infrastructore needs discussion stale

Comments

@virkt25
Copy link
Contributor

virkt25 commented Feb 9, 2018

Type

  • Question
  • Suggestion
  • Feature
  • Bug / Issue

Description / Steps to Reproduce / Suggestion

This is a follow up issue from #943 (comment)

I would like to propose that to get more community involvement we should follow the all-contributors standards ... basically means we recognize all contributions big and small (not just code PRs). This includes answering questions, design help, feature suggestions, bug reports, blogs, documentation, talks, tutorials, etc. etc. etc.

The contributors will be listed in a CONTRIBUTORS.md file at the top level of the repo. There is a tool that can help track GitHub contributions (and it provides a CLI interface for adding other contributors (blogs, talks, tutorials, etc.)). The tool formats the page so it looks good and makes it easy to update / maintain.

Reference Links

@virkt25 virkt25 added the developer-experience Issues affecting ease of use and overall experience of LB users label Feb 9, 2018
@dhmlau dhmlau added the non-DP3 label Apr 19, 2018
@dhmlau dhmlau removed the non-DP3 label Aug 23, 2018
@bajtos bajtos added Internal Tooling Issues related to our tooling and monorepo infrastructore and removed developer-experience Issues affecting ease of use and overall experience of LB users labels Jan 24, 2019
@dhmlau
Copy link
Member

dhmlau commented May 30, 2019

From @bajtos: https://github.blog/changelog/2019-05-23-triage-and-maintain-roles-beta/.
There are users/contributors who have done a pretty good job in answering others' issues. We can use the new "Triage" role to recognize them.

@achrinza
Copy link
Member

achrinza commented Feb 22, 2020

+1 for using All Contributors

Currently, the community contributions can only be viewed from GitHub contributor graph. This may be a bit too obscure, especially since it sorts by commit count by default.

I think that utilizing the @all-contributors bot will help increase visibility of community contributions:

image

Inside README:

image

This will encourage community contributors by increasing visibility of their contributions in a high-visibility section (README).

@dhmlau
Copy link
Member

dhmlau commented Feb 27, 2020

+1
Does it show all the contributors (since the beginning of this repo)? If yes, it will be 100+. It might be good just to show the pictures? I saw it in some repos but forgot where it is.

@strongloop/loopback-maintainers, WDYT?

@frbuceta
Copy link
Contributor

+1
Does it show all the contributors (since the beginning of this repo)? If yes, it will be 100+. It might be good just to show the pictures? I saw it in some repos but forgot where it is.

@strongloop/loopback-maintainers, WDYT?

Something like this?
https://frbuceta.github.io/restify-jwt-community/#contributing
https://github.com/frbuceta/restify-jwt-community/blob/master/docs/index.md

@nabdelgadir
Copy link
Contributor

@achrinza achrinza mentioned this issue Feb 27, 2020
7 tasks
@achrinza
Copy link
Member

I've created draft PR #4756 to show how it'd look like on this repo's README

@dougal83
Copy link
Contributor

dougal83 commented Feb 27, 2020

I like the first one https://frbuceta.github.io/restify-jwt-community/#contributing, although since there are 100+ contributors, would it be too cluttered?

+1 to preference for first example. I think this is one situation that looking cluttered and busy might be a good thing; demonstrating the volume of people deciding to get involved. Can we excluded bots?

@dhmlau
Copy link
Member

dhmlau commented Feb 27, 2020

Something like this?
https://frbuceta.github.io/restify-jwt-community/#contributing
https://github.com/frbuceta/restify-jwt-community/blob/master/docs/index.md

Maybe it's just me. It shows broken images for me on the 2nd link.

@dougal83, any reason you don't want bot? I think it would be ideal to have bot to update the contributors for us, because it's hard to keep track manually.

@frbuceta
Copy link
Contributor

Interesting, the truth is that it has never occurred to me to delete the bots

@dougal83
Copy link
Contributor

dougal83 commented Feb 27, 2020

@dougal83, any reason you don't want bot? I think it would be ideal to have bot to update the contributors for us, because it's hard to keep track manually.

@dhmlau A bot to update the list is fine. I'm just anti bot when it comes to their inclusion on a contribution list. 😄 Just a nice to have... I see bots as noise in this context.

@stale
Copy link

stale bot commented Dec 25, 2020

This issue has been marked stale because it has not seen activity within six months. If you believe this to be in error, please contact one of the code owners, listed in the CODEOWNERS file at the top-level of this repository. This issue will be closed within 30 days of being stale.

@stale stale bot added the stale label Dec 25, 2020
@stale
Copy link

stale bot commented Jul 14, 2021

This issue has been closed due to continued inactivity. Thank you for your understanding. If you believe this to be in error, please contact one of the code owners, listed in the CODEOWNERS file at the top-level of this repository.

@stale stale bot closed this as completed Jul 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Tooling Issues related to our tooling and monorepo infrastructore needs discussion stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants