Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Pep8, pyflakes and a better output format #4

Merged
merged 2 commits into from Nov 24, 2011

Conversation

Projects
None yet
2 participants
Contributor

kailashnath commented Nov 21, 2011

Hi,
Could you please review the diff and accept it into your main repo if you are okay with it. Please let me know if you want me to make more changes or reject some.

Changelog:

  • Made sure the file is pep8 and pyflakes compliant
  • Separated the code blocks as the function 'handle' was doing many things like running tests and printing the output which it shouldn't have been doing
  • Moved the imports regarding the template to the top of the file and made one common function as it was breaking the DRY principles

Cheers
Kailash K

@theju theju added a commit that referenced this pull request Nov 24, 2011

@theju theju Merge pull request #4 from kailashnath/master
Pep8, pyflakes and a better output format
6d9b6a2

@theju theju merged commit 6d9b6a2 into agiliq:master Nov 24, 2011

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