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 a "Hall of Fame" to the contributors list #481

Merged
merged 1 commit into from
Aug 30, 2018
Merged

Conversation

ejmr
Copy link
Collaborator

@ejmr ejmr commented Aug 23, 2018

This patch makes use of

https://github.com/sourcerer-io/hall-of-fame

to display a Hall of Fame for the top contributors. There are two
purposes to this addition:

  1. To show gratitude.
  2. To hopefully encourage more contributions.

Admittedly there is some concern this may invite developers to
"game" the system, i.e. trying "win" the Hall of Fame by submitting
more patches. In other words, it could invite a problem of quantity
over quality. However, I have fath that the emacs-php community is
more mature than that.

This patch does not update the Japanese version of the README since
it explicitly links to the list of contributors in the English version.

Special-thanks: @emacs-php
Special-thanks: Every contributor ever
Signed-off-by: Eric James Michael Ritz [email protected]

This patch makes use of

	https://github.com/sourcerer-io/hall-of-fame

to display a Hall of Fame for the top contributors.  There are two
purposes to this addition:

1. To show gratitude.
2. To hopefully encourage more contributions.

Admittedly there is some concern this may invite developers to
"game" the system, i.e. trying "win" the Hall of Fame by submitting
more patches.  In other words, it could invite a problem of quantity
over quality.  However, I have fath that the emacs-php community is
more mature than that.

This patch does not update the Japanese version of the README since
it explicitly links to the list of contributors in the English version.

Special-thanks: @emacs-php
Special-thanks: Every contributor ever
Signed-off-by: Eric James Michael Ritz <[email protected]>
@ejmr
Copy link
Collaborator Author

ejmr commented Aug 23, 2018

Note that it may take at least an hour for the content to appear in the README, due to the large number of contributions to process.

@ejmr
Copy link
Collaborator Author

ejmr commented Aug 23, 2018

This is not having the intended effect so clearly I misunderstood something about the process.

@zonuexe
Copy link
Member

zonuexe commented Aug 24, 2018

Well, I see that the code you obtained is not working.

https://github.com/emacs-php/php-mode/blob/hall-of-fame/README.md

I also got it, but it is the same.

[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/0)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/0)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/1)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/1)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/2)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/2)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/3)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/3)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/4)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/4)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/5)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/5)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/6)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/6)[![](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/images/7)](https://sourcerer.io/fame/zonuexe/emacs-php/php-mode/links/7)

@zonuexe zonuexe merged commit 1f04813 into master Aug 30, 2018
@zonuexe zonuexe deleted the hall-of-fame branch August 30, 2018 02:39
@zonuexe zonuexe mentioned this pull request Dec 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants