<div dir="ltr">Seems like there's a disagreement between yapf and flake on styling.<div>See <a href="http://jenkins.ovirt.org/job/lago_master_check-patch-el7-x86_64/362/console">http://jenkins.ovirt.org/job/lago_master_check-patch-el7-x86_64/362/console</a> and </div><div><a href="http://jenkins.ovirt.org/job/lago_master_check-patch-el7-x86_64/361/console">http://jenkins.ovirt.org/job/lago_master_check-patch-el7-x86_64/361/console</a> :<br></div><div><br></div><div><br></div><div><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(51,51,51);font-size:13px"><span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">7:24:57</b> </span>scripts/check_style.sh
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:24:57</b> </span>yapf 0.7.1
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>--- lago/templates.py (original)
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>+++ lago/templates.py (reformatted)
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>@@ -260,7 +260,6 @@
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> finally:
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> response.close()
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>-
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> #: Registry for template providers
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> _PROVIDERS = {
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> 'file': FileSystemTemplateProvider,
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span>Yapf failed, make sure to run:
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:25:59</b> </span> yapf --style .style.yapf --in-place --recursive .</pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(51,51,51);font-size:13px"><br></pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(51,51,51);font-size:13px">vs.</pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(51,51,51);font-size:13px"><br></pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px;color:rgb(51,51,51);font-size:13px"><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px"><span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:13:48</b> </span>PYTHONPATH=/home/jenkins/workspace/lago_master_check-patch-el7-x86_64/lago python /home/jenkins/workspace/lago_master_check-patch-el7-x86_64/lago/tests/venv/bin/flake8
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:13:52</b> </span>./lago/templates.py:264:1: E305 expected 2 blank lines after class or function definition, found 1
<span class="gmail-timestamp" style="box-sizing:border-box"><b style="box-sizing:border-box">07:13:52</b> </span>make: *** [check-local] Error 1</pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px"><br></pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px"><br></pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px">Come on... I just want my patch in, it has nothing to do with this supposed breakage.</pre><pre class="gmail-console-output" style="box-sizing:border-box;white-space:pre-wrap;word-wrap:break-word;margin-top:0px;margin-bottom:0px">Y.</pre></pre></div></div>