From ca614beac749f141eedb7b8d54ac84c6b6e4fe8d Mon Sep 17 00:00:00 2001
From: Felipe Monteiro <felipe.monteiro@att.com>
Date: Sun, 25 Nov 2018 12:29:24 -0500
Subject: [PATCH] trivial: fix yapf/pep8 interaction failing on logical
 operator

This adds a custom yapf config file with
split_before_logical_operator knob set to false so that
pep8 doesn't complain when:

    if (save_location and save_location != os.path.sep and
            save_location.endswith(os.path.sep)):

is wrongly converted to:

    if (save_location and save_location != os.path.sep
            and save_location.endswith(os.path.sep)):

This keeps the and on the first line as in the first example.

Change-Id: Ibf2813c3c969fd5f1cfd14936f3fecc9100283f7
---
 .style.yapf | 5 +++++
 tox.ini     | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)
 create mode 100644 .style.yapf

diff --git a/.style.yapf b/.style.yapf
new file mode 100644
index 00000000..4f3489d0
--- /dev/null
+++ b/.style.yapf
@@ -0,0 +1,5 @@
+[style]
+based_on_style = pep8
+spaces_before_comment = 2
+column_limit = 79
+split_before_logical_operator = false
diff --git a/tox.ini b/tox.ini
index 14bb2e3b..14c6df7f 100644
--- a/tox.ini
+++ b/tox.ini
@@ -24,7 +24,7 @@ basepython = python3
 deps =
     -r{toxinidir}/test-requirements.txt
 commands =
-    yapf --style=pep8 -ir {toxinidir}/pegleg {toxinidir}/tests
+    yapf -ir {toxinidir}/pegleg {toxinidir}/tests
 
 [testenv:pep8]
 basepython = python3