search

abiczo / github-notifier

Github updates notifier for Linux
57 stars 8 forks source link

Updates of organizations #13

Closed thotmx closed 12 years ago

thotmx commented 12 years ago

This code allow to get the updates of the user's organizations. I needed this feature, and add the code to do that. Take a look, I hope this will be useful.

abiczo commented 12 years ago

Great thanks! I'm going to comment on a couple of nits but looks good otherwise!

abiczo commented 12 years ago

Nice work, really appreciated! Sorry about the lots of nits. Just one more general comment: I think some of the lines may go over 79 chars, as per pep8 any chance you could try to break those up into multiple lines?

Thanks again!

thotmx commented 12 years ago

Don't worry about it. I will fix it in the next commit. Thaks a lot for your comments, are really good to make a better code.

thotmx commented 12 years ago

I think that's all. I don't know how to break some lines in 79 chars.

abiczo commented 12 years ago

Awesome, thanks for the updates! This is looking good.

abiczo commented 12 years ago

Also thanks for updating the api calls to use the latest api version.