Hashar has uploaded a new change for review.
https://gerrit.wikimedia.org/r/262597
Change subject: varnish: lint varnishlog.py
......................................................................
varnish: lint varnishlog.py
E226 missing whitespace around arithmetic operator
E123 closing bracket does not match indentation of opening bracket's
line
Part of migrating to plain pep8 T114887
Change-Id: I250709c43031ffbdac0a95ff927cb76da3102b4a
---
M modules/varnish/files/varnishlog.py
1 file changed, 3 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/operations/puppet
refs/changes/97/262597/1
diff --git a/modules/varnish/files/varnishlog.py
b/modules/varnish/files/varnishlog.py
index ab9c968..e218a99 100755
--- a/modules/varnish/files/varnishlog.py
+++ b/modules/varnish/files/varnishlog.py
@@ -138,11 +138,11 @@
arg = args[i].translate(None, '-')
# If this is the end, or if the next arg has a '-',
# assume this is a switch that does not take a parameter.
- if i == len(args) - 1 or '-' in args[i+1]:
+ if i == len(args) - 1 or '-' in args[i + 1]:
arg_tuple = (arg, '')
# Else this arg takes a parameter.
else:
- arg_tuple = (arg, args[i+1])
+ arg_tuple = (arg, args[i + 1])
i += 1
i += 1
# The key in the VSL arg tuple must be a single character.
@@ -166,8 +166,7 @@
tag.ljust(12),
remote_party,
value
- )
- )
+ ))
if __name__ == '__main__':
--
To view, visit https://gerrit.wikimedia.org/r/262597
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I250709c43031ffbdac0a95ff927cb76da3102b4a
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
