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

Update __init__.py #21

Merged
merged 2 commits into from Jul 17, 2013

Conversation

Projects
None yet
3 participants
@haneefmubarak
Copy link
Contributor

haneefmubarak commented Jul 15, 2013

Added various things here and there, fixed issue #14.

Update __init__.py
Added various things here and there, fixed issue #14.
@@ -103,8 +103,11 @@
"Python": "Pythonista",
"Ruby": "Rubyist",
"Go": "Gopher",
"Java": "Javavore",
"C": "sysadmin",
"FORTRAN": "old-school hacker",

This comment has been minimized.

@michaelficarra

This comment has been minimized.

@haneefmubarak

haneefmubarak Jul 15, 2013

Contributor

Oops. I can fix that.

@@ -345,7 +352,7 @@ def get_stats(username):
if len(events):
if events[0] in evtactions:
summary += (" who <a href=\"#events\">would rather be {0} "
"instead of pushing code</a>").format(
"instead of pushing code, but still pushes code</a>").format(

This comment has been minimized.

@michaelficarra

michaelficarra Jul 15, 2013

this sounds awkward

This comment has been minimized.

@haneefmubarak

haneefmubarak Jul 15, 2013

Contributor

I know. But it's the only wording I could think of to resolve #14

Update __init__.py
fixed double space on line 108
@haneefmubarak

This comment has been minimized.

Copy link
Contributor

haneefmubarak commented Jul 15, 2013

fixed double space on FORTRAN line

@dfm

This comment has been minimized.

Copy link
Owner

dfm commented Jul 16, 2013

Thanks for this! I also think that the wording is a little funny but I'll try to play with it a bit more when I merge.

Thanks again!

🍻

@haneefmubarak

This comment has been minimized.

Copy link
Contributor

haneefmubarak commented Jul 16, 2013

No problem. Also, it would be cool if the demo site actually ran full-fledged and updated itself every day? Should I open an issue?

@dfm

This comment has been minimized.

Copy link
Owner

dfm commented Jul 16, 2013

Agreed. You can try but no promises… pull requests are always better 😉

@dfm dfm merged commit 62a529f into dfm:master Jul 17, 2013

@haneefmubarak haneefmubarak deleted the haneefmubarak:patch-1 branch Jul 18, 2013

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