Dzahn has submitted this change and it was merged. Change subject: Revert "Get rid of .pep8 files" ......................................................................
Revert "Get rid of .pep8 files" It is too soon, we will want to: 1) support tox via: https://gerrit.wikimedia.org/r/244148 2) switch CI to it and get rid of the current pep8 job 3) get rid of the .pep8 files Meanwhile, that is breaking pep8 job. This reverts commit 07c62899fa052fdd6af222e20ce20fcc8fe51286. Bug: T114887 Change-Id: I47a79917ec1d42b6a12821eef076dc0216597544 --- A files/misc/.pep8 A modules/dataset/files/.pep8 A modules/dnsrecursor/files/.pep8 A modules/dynamicproxy/files/.pep8 A modules/eventlogging/files/.pep8 A modules/ganglia/files/plugins/.pep8 A modules/icinga/files/.pep8 A modules/ldap/files/scripts/.pep8 A modules/librenms/files/.pep8 A modules/memcached/files/ganglia/.pep8 A modules/mysql_wmf/files/ganglia/plugins/.pep8 A modules/openstack/files/juno/nova/.pep8 A modules/openstack/files/juno/virtscripts/.pep8 A modules/openstack/files/kilo/virtscripts/.pep8 A modules/osm/files/ganglia/.pep8 A modules/postgresql/files/ganglia/.pep8 A modules/puppetmaster/files/.pep8 A modules/redis/files/ganglia/.pep8 A modules/servermon/files/.pep8 A modules/swift/files/.pep8 A modules/swift/files/SwiftMedia/wmf/.pep8 A modules/toollabs/files/.pep8 A modules/varnish/files/ganglia/.pep8 A utils/.pep8 24 files changed, 105 insertions(+), 0 deletions(-) Approvals: jenkins-bot: Verified Dzahn: Looks good to me, approved diff --git a/files/misc/.pep8 b/files/misc/.pep8 new file mode 100644 index 0000000..ad4323f --- /dev/null +++ b/files/misc/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/dataset/files/.pep8 b/modules/dataset/files/.pep8 new file mode 100644 index 0000000..ad4323f --- /dev/null +++ b/modules/dataset/files/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/dnsrecursor/files/.pep8 b/modules/dnsrecursor/files/.pep8 new file mode 100644 index 0000000..bcc4cdd --- /dev/null +++ b/modules/dnsrecursor/files/.pep8 @@ -0,0 +1,6 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +# E241: multiple spaces after ',' +ignore = E501,E241 diff --git a/modules/dynamicproxy/files/.pep8 b/modules/dynamicproxy/files/.pep8 new file mode 100644 index 0000000..f225d9e --- /dev/null +++ b/modules/dynamicproxy/files/.pep8 @@ -0,0 +1,3 @@ +[pep8] +# 80 cols is too short +max-line-length=100 diff --git a/modules/eventlogging/files/.pep8 b/modules/eventlogging/files/.pep8 new file mode 100644 index 0000000..609e631 --- /dev/null +++ b/modules/eventlogging/files/.pep8 @@ -0,0 +1,6 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +# E241: multiple spaces after ',' +ignore = E501,E241 diff --git a/modules/ganglia/files/plugins/.pep8 b/modules/ganglia/files/plugins/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/ganglia/files/plugins/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/icinga/files/.pep8 b/modules/icinga/files/.pep8 new file mode 100644 index 0000000..ad4323f --- /dev/null +++ b/modules/icinga/files/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/ldap/files/scripts/.pep8 b/modules/ldap/files/scripts/.pep8 new file mode 100644 index 0000000..baa77a4 --- /dev/null +++ b/modules/ldap/files/scripts/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/librenms/files/.pep8 b/modules/librenms/files/.pep8 new file mode 100644 index 0000000..bd39b63 --- /dev/null +++ b/modules/librenms/files/.pep8 @@ -0,0 +1,3 @@ +[pep8] +# This is a third-party file, so I'm just turning off everything... +ignore = W191,E225,E501,W239,E303,W209,E302,W293,E101,E128,E121,E703,W291,E401,E701,E221,E211,E203 diff --git a/modules/memcached/files/ganglia/.pep8 b/modules/memcached/files/ganglia/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/memcached/files/ganglia/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/mysql_wmf/files/ganglia/plugins/.pep8 b/modules/mysql_wmf/files/ganglia/plugins/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/mysql_wmf/files/ganglia/plugins/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/openstack/files/juno/nova/.pep8 b/modules/openstack/files/juno/nova/.pep8 new file mode 100644 index 0000000..ad4323f --- /dev/null +++ b/modules/openstack/files/juno/nova/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/openstack/files/juno/virtscripts/.pep8 b/modules/openstack/files/juno/virtscripts/.pep8 new file mode 100644 index 0000000..542c110 --- /dev/null +++ b/modules/openstack/files/juno/virtscripts/.pep8 @@ -0,0 +1,3 @@ +[pep8] +# This is a third-party file, so I'm just turning off everything... +ignore = W191,E225,E501,W239,E303,W209,E302,W293,E101,E128,E121,E703,W291 diff --git a/modules/openstack/files/kilo/virtscripts/.pep8 b/modules/openstack/files/kilo/virtscripts/.pep8 new file mode 100644 index 0000000..77f5a1c --- /dev/null +++ b/modules/openstack/files/kilo/virtscripts/.pep8 @@ -0,0 +1,3 @@ +[pep8] +# This is a third-party file, so I'm just turning off everything... +ignore = W191,E225,E501,W239,E303,W209,E302,W293,E101,E128,E121,E703,W291,E131,E265 diff --git a/modules/osm/files/ganglia/.pep8 b/modules/osm/files/ganglia/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/osm/files/ganglia/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/postgresql/files/ganglia/.pep8 b/modules/postgresql/files/ganglia/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/postgresql/files/ganglia/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/puppetmaster/files/.pep8 b/modules/puppetmaster/files/.pep8 new file mode 100644 index 0000000..baa77a4 --- /dev/null +++ b/modules/puppetmaster/files/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 diff --git a/modules/redis/files/ganglia/.pep8 b/modules/redis/files/ganglia/.pep8 new file mode 100644 index 0000000..93c1664 --- /dev/null +++ b/modules/redis/files/ganglia/.pep8 @@ -0,0 +1,4 @@ +[pep8] + +# These files are all from upstream. So, just ignore them. +exclude=* diff --git a/modules/servermon/files/.pep8 b/modules/servermon/files/.pep8 new file mode 100644 index 0000000..de50795 --- /dev/null +++ b/modules/servermon/files/.pep8 @@ -0,0 +1,2 @@ +[pep8] +ignore = E128,E127,E241 diff --git a/modules/swift/files/.pep8 b/modules/swift/files/.pep8 new file mode 100644 index 0000000..16c865a --- /dev/null +++ b/modules/swift/files/.pep8 @@ -0,0 +1,6 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +# E221: multiple spaces before operator +ignore = E501,E241 diff --git a/modules/swift/files/SwiftMedia/wmf/.pep8 b/modules/swift/files/SwiftMedia/wmf/.pep8 new file mode 100644 index 0000000..c3efd5d --- /dev/null +++ b/modules/swift/files/SwiftMedia/wmf/.pep8 @@ -0,0 +1,6 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +# E221: multiple spaces before operator +ignore = E501,E221 diff --git a/modules/toollabs/files/.pep8 b/modules/toollabs/files/.pep8 new file mode 100644 index 0000000..609e631 --- /dev/null +++ b/modules/toollabs/files/.pep8 @@ -0,0 +1,6 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +# E241: multiple spaces after ',' +ignore = E501,E241 diff --git a/modules/varnish/files/ganglia/.pep8 b/modules/varnish/files/ganglia/.pep8 new file mode 100644 index 0000000..2f6357a --- /dev/null +++ b/modules/varnish/files/ganglia/.pep8 @@ -0,0 +1,2 @@ +[pep8] +exclude=* diff --git a/utils/.pep8 b/utils/.pep8 new file mode 100644 index 0000000..ad4323f --- /dev/null +++ b/utils/.pep8 @@ -0,0 +1,5 @@ +[pep8] + +# Suppress warnings: +# E501: line too long +ignore = E501 -- To view, visit https://gerrit.wikimedia.org/r/263925 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47a79917ec1d42b6a12821eef076dc0216597544 Gerrit-PatchSet: 2 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Hashar <[email protected]> Gerrit-Reviewer: BBlack <[email protected]> Gerrit-Reviewer: Dzahn <[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
