From 1a30d3fcfdce31995c669d2533ac8037533eb56a Mon Sep 17 00:00:00 2001 From: Ben Mather Date: Tue, 30 Sep 2014 08:07:40 +0100 Subject: [PATCH] mark list as private instead of shadowing --- lib/ansible/modules/extras/source_control/github_hooks.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/ansible/modules/extras/source_control/github_hooks.py b/lib/ansible/modules/extras/source_control/github_hooks.py index 0ca58796d13..878e3249afe 100644 --- a/lib/ansible/modules/extras/source_control/github_hooks.py +++ b/lib/ansible/modules/extras/source_control/github_hooks.py @@ -75,7 +75,7 @@ EXAMPLES = ''' - local_action: github_hooks action=cleanall user={{ gituser }} oauthkey={{ oauthkey }} repo={{ repo }} ''' -def list_(module, hookurl, oauthkey, repo, user): +def _list(module, hookurl, oauthkey, repo, user): url = "%s/hooks" % repo auth = base64.encodestring('%s:%s' % (user, oauthkey)).replace('\n', '') headers = { @@ -88,7 +88,7 @@ def list_(module, hookurl, oauthkey, repo, user): return False, response.read() def clean504(module, hookurl, oauthkey, repo, user): - current_hooks = list_(hookurl, oauthkey, repo, user)[1] + current_hooks = _list(hookurl, oauthkey, repo, user)[1] decoded = json.loads(current_hooks) for hook in decoded: @@ -100,7 +100,7 @@ def clean504(module, hookurl, oauthkey, repo, user): return 0, current_hooks def cleanall(module, hookurl, oauthkey, repo, user): - current_hooks = list_(hookurl, oauthkey, repo, user)[1] + current_hooks = _list(hookurl, oauthkey, repo, user)[1] decoded = json.loads(current_hooks) for hook in decoded: @@ -162,7 +162,7 @@ def main(): content_type = module.params['content_type'] if action == "list": - (rc, out) = list_(module, hookurl, oauthkey, repo, user) + (rc, out) = _list(module, hookurl, oauthkey, repo, user) if action == "clean504": (rc, out) = clean504(module, hookurl, oauthkey, repo, user)