search

jlafon / ansible-profile

An Ansible plugin for timing tasks
http://jlafon.io/ansible-profiling.html
MIT License
342 stars 57 forks source link

Update profile_tasks.py #7

Closed aioue closed 9 years ago

aioue commented 9 years ago

Extract duplicate code to function.

aioue commented 9 years ago

any changes you need to this @jlafon. Happy to close if you're not accepting PRs

jlafon commented 9 years ago

Sorry for the slow reply. Is there any reason that you didn't put the method in the class?

aioue commented 9 years ago

Nope. Blindly copying the style of https://github.com/junaid18183/ansible_home/blob/master/ansible_plugins/callback_plugins/timestamp.py.old

jlafon commented 9 years ago

Would you mind putting the method on the class and updating the PR? I can do it otherwise.

aioue commented 9 years ago

Mashup/modified plugin merged into v2 release of Ansible. Will close this PR and direct further improvements there. Thanks @jlafon!