jenkins-bot has submitted this change and it was merged. Change subject: Use tox for running flake8 ......................................................................
Use tox for running flake8 Also instead of totally ignoring line length, set a max of 120 The [pep8] section is left in since jenkins is still running pep8, and should be removed once jenkins is updated to use tox instead. Change-Id: Id18ae52a5db4d1d7cb112fde895dde94217d38da --- M .gitignore M tox.ini 2 files changed, 18 insertions(+), 2 deletions(-) Approvals: Hashar: Looks good to me, approved jenkins-bot: Verified diff --git a/.gitignore b/.gitignore index 6625c5d..19649be 100644 --- a/.gitignore +++ b/.gitignore @@ -7,3 +7,4 @@ tests/browser/.gem tests/browser/.bundle node_modules +.tox diff --git a/tox.ini b/tox.ini index 555e4de..d432c1b 100644 --- a/tox.ini +++ b/tox.ini @@ -1,6 +1,21 @@ +[tox] + +# Ensure 1.6+ is used to support 'skipsdist' +minversion = 1.6 + +# Do not run install command +skipsdist = True + +# Environements to execute when invoking 'tox' +envlist = flake8 + +[testenv:flake8] +commands = flake8 +deps = flake8 + [flake8] -; E501 line too long -ignore = E501 +exclude = .tox +max_line_length = 120 [pep8] ; E501 line too long -- To view, visit https://gerrit.wikimedia.org/r/196744 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id18ae52a5db4d1d7cb112fde895dde94217d38da Gerrit-PatchSet: 3 Gerrit-Project: mediawiki/extensions/UploadWizard Gerrit-Branch: master Gerrit-Owner: Legoktm <[email protected]> Gerrit-Reviewer: Gergő Tisza <[email protected]> Gerrit-Reviewer: Gilles <[email protected]> Gerrit-Reviewer: Hashar <[email protected]> Gerrit-Reviewer: Legoktm <[email protected]> Gerrit-Reviewer: MarkTraceur <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
