Skip to content

Conversation

@techtonik
Copy link
Contributor

This may encourage more Python folks to send edits.

docs/conf.py Outdated
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