From 172a49584a3e55f6eaea7aff2f9f04ef54f9ed02 Mon Sep 17 00:00:00 2001 From: Jiri Podivin Date: Fri, 22 Jan 2021 13:46:56 +0100 Subject: [PATCH] Style adjustment of a constant. Perhaps slight exaggeration. The value is set just after imports, depending on a single condition, but before declaring anything else. Thus making it quasi-constant. Still, it is beneficial from readability perspective. Signed-off-by: Jiri Podivin Change-Id: Ieddc2cabd2612fa406a9660daa17a836ea1ee4cf --- validations_libs/ansible.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/validations_libs/ansible.py b/validations_libs/ansible.py index 926c6a03..4a7d9797 100644 --- a/validations_libs/ansible.py +++ b/validations_libs/ansible.py @@ -39,9 +39,9 @@ except NameError: try: version = pkg_resources.get_distribution("ansible_runner").version - backward_compat = (version < '1.4.0') + BACKWARD_COMPAT = (version < '1.4.0') except pkg_resources.DistributionNotFound: - backward_compat = False + BACKWARD_COMPAT = False class Ansible(object): @@ -433,7 +433,7 @@ class Ansible(object): 'ident': '' } - if not backward_compat: + if not BACKWARD_COMPAT: r_opts.update({ 'envvars': envvars, 'project_dir': playbook_dir, @@ -458,7 +458,7 @@ class Ansible(object): runner_config.prepare() runner_config.env['ANSIBLE_STDOUT_CALLBACK'] = \ envvars['ANSIBLE_STDOUT_CALLBACK'] - if backward_compat: + if BACKWARD_COMPAT: runner_config.env.update(envvars) runner = ansible_runner.Runner(config=runner_config)