Andrew Bogott has submitted this change and it was merged.
Change subject: toollabs: lint genpp.py
......................................................................
toollabs: lint genpp.py
E226 missing whitespace around arithmetic operator
Part of migrating to plain pep8 T114887
Change-Id: I4577782d747b8fc31b613ebf0a861e1553395c3f
---
M modules/toollabs/manifests/genpp/genpp.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Andrew Bogott: Looks good to me, approved
Merlijn van Deen: Looks good to me, but someone else must approve
jenkins-bot: Verified
diff --git a/modules/toollabs/manifests/genpp/genpp.py
b/modules/toollabs/manifests/genpp/genpp.py
index c90726d..8b76ead 100644
--- a/modules/toollabs/manifests/genpp/genpp.py
+++ b/modules/toollabs/manifests/genpp/genpp.py
@@ -21,7 +21,7 @@
keep_trailing_newline=True
)
-env.filters['pline'] = lambda x: repr(x) + "," + (" " * (20-len(x)))
+env.filters['pline'] = lambda x: repr(x) + "," + (" " * (20 - len(x)))
env.filters['pydictsort'] = lambda x: \
sorted(x.items(), key=lambda y: (y[0].split('-', 1)[1].lower(), y))
--
To view, visit https://gerrit.wikimedia.org/r/262596
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4577782d747b8fc31b613ebf0a861e1553395c3f
Gerrit-PatchSet: 3
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <[email protected]>
Gerrit-Reviewer: Andrew Bogott <[email protected]>
Gerrit-Reviewer: Chasemp <[email protected]>
Gerrit-Reviewer: Hashar <[email protected]>
Gerrit-Reviewer: John Vandenberg <[email protected]>
Gerrit-Reviewer: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: Yuvipanda <[email protected]>
Gerrit-Reviewer: coren <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
