diff --git a/deckhand/common/utils.py b/deckhand/common/utils.py index c606c8b3..1f0f142f 100644 --- a/deckhand/common/utils.py +++ b/deckhand/common/utils.py @@ -288,8 +288,8 @@ def deepfilter(dct, **filters): # Else if both the filter value and the actual value in the doc # are dictionaries, check whether the filter dict is a subset # of the actual dict. - if (isinstance(actual_val, dict) - and isinstance(filter_val, dict)): + if (isinstance(actual_val, dict) and + isinstance(filter_val, dict)): is_subset = set( filter_val.items()).issubset(set(actual_val.items())) if not is_subset: diff --git a/deckhand/engine/layering.py b/deckhand/engine/layering.py index bc04c506..be109ddf 100644 --- a/deckhand/engine/layering.py +++ b/deckhand/engine/layering.py @@ -563,8 +563,8 @@ class DocumentLayering(object): parent_name=overall_data.name, action=action) - if (isinstance(from_parent, dict) - and isinstance(from_child, dict)): + if (isinstance(from_parent, dict) and + isinstance(from_child, dict)): engine_utils.deep_merge(from_parent, from_child) if from_parent is not None: diff --git a/deckhand/engine/secrets_manager.py b/deckhand/engine/secrets_manager.py index fa3079a3..09c57d5f 100644 --- a/deckhand/engine/secrets_manager.py +++ b/deckhand/engine/secrets_manager.py @@ -402,8 +402,8 @@ class SecretsSubstitution(object): substituted_data = utils.jsonpath_replace( document['data'], src_secret, dest_path, dest_pattern) - if (isinstance(document['data'], dict) - and isinstance(substituted_data, dict)): + if (isinstance(document['data'], dict) and + isinstance(substituted_data, dict)): document['data'].update(substituted_data) elif substituted_data: document['data'] = substituted_data diff --git a/deckhand/engine/utils.py b/deckhand/engine/utils.py index b6d11044..a8fbf787 100644 --- a/deckhand/engine/utils.py +++ b/deckhand/engine/utils.py @@ -29,8 +29,8 @@ def deep_merge(dct, merge_dct): :return: None """ for k, v in merge_dct.items(): - if (k in dct and isinstance(dct[k], dict) - and isinstance(merge_dct[k], collections.Mapping)): + if (k in dct and isinstance(dct[k], dict) and + isinstance(merge_dct[k], collections.Mapping)): deep_merge(dct[k], merge_dct[k]) else: dct[k] = merge_dct[k]