Skip to content

Give credits #343

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

Closed
wants to merge 4 commits into from
Closed

Give credits #343

wants to merge 4 commits into from

Conversation

techtonik
Copy link
Contributor

This may encourage more Python folks to send edits.

@@ -46,7 +46,7 @@

# General information about the project.
project = u'pythonguide'
copyright = u'2013. A <a href="http://kennethreitz.com/pages/open-projects.html">Kenneth Reitz</a> Project. <a href="http://creativecommons.org/licenses/by-nc-sa/3.0/"> Creative Commons Share-Alike 3.0</a>'
copyright = u'2013. Project by <a href="http://kennethreitz.com/pages/open-projects.html">Kenneth Reitz</a> and <a href="https://github.com/kennethreitz/python-guide/graphs/contributors">Python folks</a>. <a href="http://creativecommons.org/licenses/by-nc-sa/3.0/"> Creative Commons Share-Alike 3.0</a>'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about "and contributors" instead?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 for "and contributors"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed. ✅

@sigmavirus24
Copy link
Contributor

@techtonik did you sign the CLA? Just kidding

@techtonik
Copy link
Contributor Author

@sigmavirus24, no, passed Most Valuable Patent Troll certification instead. ;)

@kennethreitz
Copy link
Contributor

Love this! But, let's find another place for this. "A Kenneth Reitz Project" is a special phrase I put at the bottom of all my projects :)

@techtonik
Copy link
Contributor Author

Well, makes sense to leave only bus factor subjects there. =)

@sigmavirus24
Copy link
Contributor

@techtonik can you come up with an alternate wording for this to satisfy @kennethreitz that'd be awesome. I'd love to merge this.

@techtonik
Copy link
Contributor Author

@kennethreitz, I restored the special phrase. Is that ok? =)

@sigmavirus24
Copy link
Contributor

@kennethreitz this preserves your "A Kenneth Reitz project" and adds the credit given to contributors. I'm 👍, can you give it one last once over?

@kennethreitz
Copy link
Contributor

You guys have no eye for aesthetic :)

Let's put a link in the sidebar.

@sigmavirus24
Copy link
Contributor

Damnit Jim I'm a programmer not a designer!

(Self-portrait for reference: McCoy)

@kennethreitz
Copy link
Contributor

Replace the entire Feedback section with Contributors, including a rough estimation of the number of people who have sent pull requests :)

@techtonik
Copy link
Contributor Author

This will require fetching and parsing https://api.github.com/repos/kennethreitz/python-guide/stats/contributors on each request, so unless some HTML5 guru jumps in to help with caching info to local storage based on last guide hash, I don't really see the way how to keep that updated.

http://developer.github.com/v3/repos/statistics/

It is also possible that we will hit Github rate limits on API calls too soon http://developer.github.com/v3/#rate-limiting

@kennethreitz
Copy link
Contributor

No, just plain text. KISS.

@techtonik
Copy link
Contributor Author

Plain text list should still be updated somehow, so there should be script or somethings that produces this plain text.

The Github API can not be used, because it returns only 100 contributors out of current 135. Any ideas how to get the list with plain Git?

@kennethreitz
Copy link
Contributor

No, just plain text. KISS.

@techtonik
Copy link
Contributor Author

But just the block with some info about contributors and link to Github statistics page is boring. The short version of Github hi-score in the blank space of the sidebar would be more fun.

@kennethreitz
Copy link
Contributor

This is a carefully curated guide, that is comprised of text, not JavaScript. Documentation is sacred, please respect it ;)

@techtonik
Copy link
Contributor Author

tl;dr

Working on improving the patch. Stay tuned.

@techtonik
Copy link
Contributor Author

Opened a clean pull request for that.

@techtonik techtonik closed this Feb 11, 2014
@techtonik techtonik deleted the patch-1 branch May 14, 2014 19:05
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.

5 participants