Fixing code to accept "PEP8"

mozillahispano / mozbuzz

Tracking tool for media mentions

http://www.mozilla-hispano.org

15 stars 25 forks source link

Closed yograterol closed 11 years ago

yograterol commented 11 years ago

I add some changes according to PEP8 rules, and shorten "if"s

nukeador commented 11 years ago

@sveggiani r?

sebasmagri commented 11 years ago

@yograterol I've left some comments. Particularly about the shorter ifs. Please check this out and update the pull request accordingly.

Thanks and Regards!

yograterol commented 11 years ago

Listo, ya tome en cuenta tus comentarios.

sebasmagri commented 11 years ago

@yograterol please fix the commented error and update the PR.

@nukeador other than that it looks ok for me.

yograterol commented 11 years ago

@sebasmagri , all updates are ready. Thank u for your suggestions

nukeador commented 11 years ago

@sebasmagri You should be able to merge this request directly when you think it's ready.

sebasmagri commented 11 years ago

@nukeador Ok thanks!

yograterol commented 11 years ago

@sebasmagri all right?

sebasmagri commented 11 years ago

On May 9, 2013 11:49 AM, "Yohan Graterol" notifications@github.com wrote:

@sebasmagri all right?

— Reply to this email directly or view it on GitHub.

Hi! I'm sorry.

I'll merge this as soon as I'm back in home.

Regards.

sebasmagri commented 11 years ago

@nukeador it doesn't look like I have push/management rights over this repo.

Regards,