jenkins-bot has submitted this change and it was merged.
Change subject: Use flake8-putty to merge docstring rules
......................................................................
Use flake8-putty to merge docstring rules
flake8-putty allows per file ignore codes.
Change-Id: Id55b466b321357f2a6fc97ef5b93c0fee1f86d03
---
M pywikibot/_wbtypes.py
M tests/deprecation_tests.py
M tests/dry_site_tests.py
M tests/i18n_tests.py
M tests/isbn_tests.py
M tests/tools_formatter_tests.py
M tox.ini
7 files changed, 23 insertions(+), 172 deletions(-)
Approvals:
Xqt: Looks good to me, approved
jenkins-bot: Verified
diff --git a/pywikibot/_wbtypes.py b/pywikibot/_wbtypes.py
index 7d9d908..c59a5b8 100644
--- a/pywikibot/_wbtypes.py
+++ b/pywikibot/_wbtypes.py
@@ -20,6 +20,7 @@
"""Abstract class for Wikibase representations."""
def __init__(self):
+ """Constructor."""
raise NotImplementedError
def toWikibase(self):
diff --git a/tests/deprecation_tests.py b/tests/deprecation_tests.py
index 8ecc982..97a58b5 100644
--- a/tests/deprecation_tests.py
+++ b/tests/deprecation_tests.py
@@ -120,11 +120,13 @@
@remove_last_args(['foo', 'bar'])
def deprecated_all():
+ """Test remove_last_args with all args removed."""
return None
@remove_last_args(['bar'])
def deprecated_all2(foo):
+ """Test remove_last_args with one arg removed."""
return foo
diff --git a/tests/dry_site_tests.py b/tests/dry_site_tests.py
index baa5687..b047221 100644
--- a/tests/dry_site_tests.py
+++ b/tests/dry_site_tests.py
@@ -210,6 +210,7 @@
# Implemented without setUpClass(cls) and global variables as objects
# were not completely disposed and recreated but retained 'memory'
def setUp(self):
+ """Set up test method."""
super(TestNeedVersion, self).setUp()
self.version = lambda: "1.13"
diff --git a/tests/i18n_tests.py b/tests/i18n_tests.py
index 9b68faf..de85354 100644
--- a/tests/i18n_tests.py
+++ b/tests/i18n_tests.py
@@ -27,6 +27,7 @@
net = False
def setUp(self):
+ """Set up test method."""
self.msg_localized = {'en': u'test-localized EN',
'nl': u'test-localized NL',
'fy': u'test-localized FY'}
diff --git a/tests/isbn_tests.py b/tests/isbn_tests.py
index 9f69939..506b65a 100644
--- a/tests/isbn_tests.py
+++ b/tests/isbn_tests.py
@@ -179,6 +179,7 @@
@classmethod
def setUpClass(cls):
+ """Set up test class."""
super(TestIsbnWikibaseBot, cls).setUpClass()
# Check if the unit test item page and the property both exist
diff --git a/tests/tools_formatter_tests.py b/tests/tools_formatter_tests.py
index d6803f3..eb1a84b 100644
--- a/tests/tools_formatter_tests.py
+++ b/tests/tools_formatter_tests.py
@@ -38,7 +38,10 @@
class DummyUnicode(UnicodeMixin):
+ """Dummy class that __unicode__ returns a non-ascii unicode
value."""
+
def __unicode__(self):
+ """Return ä."""
return 'ä'
net = False
diff --git a/tox.ini b/tox.ini
index 219485d..a24e41d 100644
--- a/tox.ini
+++ b/tox.ini
@@ -1,16 +1,15 @@
[tox]
minversion = 1.6
skipsdist = True
-envlist = flake8,flake8-py3,flake8-docstrings-mandatory,py26,py27,py34
+envlist = flake8,flake8-py3,py26,py27,py34
[tox:jenkins]
# Override default for WM Jenkins
# Others are run in their own individual jobs on WM Jenkins
-envlist = flake8,flake8-py3,flake8-docstrings-mandatory
+envlist = flake8,flake8-py3
[params]
nose_skip = --ignore-files=(gui\.py|botirc\.py|rcstream\.py|api\.py)
-flake8_ignore = D102,D103,{[flake8]ignore}
[testenv]
setenv =
@@ -26,11 +25,12 @@
[testenv:flake8]
commands =
flake8 --version
- flake8 --ignore={[params]flake8_ignore} {posargs}
+ flake8 --ignore={[flake8]ignore} {posargs}
basepython = python2.7
deps = flake8
hacking
flake8-docstrings<0.2.2
+ flake8-putty
flake8-coding
flake8-future-import
flake8-string-format
@@ -39,177 +39,11 @@
[testenv:flake8-py3]
commands =
flake8 --version
- flake8 --ignore={[params]flake8_ignore} {posargs}
+ flake8 --ignore={[flake8]ignore} {posargs}
basepython = python3
deps = flake8
flake8-docstrings<0.2.2
-
-[testenv:flake8-docstrings]
-commands = flake8 {posargs}
-deps = flake8>=2.2.5
- flake8-docstrings<0.2.2
-
-[testenv:flake8-docstrings-mandatory]
-commands = flake8 \
- docs/conf.py \
- ez_setup.py \
- generate_user_files.py \
- pwb.py \
- pywikibot/__init__.py \
- pywikibot/backports.py \
- pywikibot/bot.py \
- pywikibot/bot_choice.py \
- pywikibot/botirc.py \
- pywikibot/comms/ \
- pywikibot/compat/ \
- pywikibot/config2.py \
- pywikibot/daemonize.py \
- pywikibot/data/ \
- pywikibot/date.py \
- pywikibot/diff.py \
- pywikibot/echo.py \
- pywikibot/editor.py \
- pywikibot/exceptions.py \
- pywikibot/family.py \
- pywikibot/flow.py \
- pywikibot/interwiki_graph.py \
- pywikibot/families/ \
- pywikibot/fixes.py \
- pywikibot/i18n.py \
- pywikibot/logentries.py \
- pywikibot/logging.py \
- pywikibot/login.py \
- pywikibot/page.py \
- pywikibot/pagegenerators.py \
- pywikibot/plural.py \
- pywikibot/proofreadpage.py \
- pywikibot/site_detect.py \
- pywikibot/site.py \
- pywikibot/textlib.py \
- pywikibot/throttle.py \
- pywikibot/titletranslate.py \
- pywikibot/tools/ \
- pywikibot/userinterfaces/ \
- pywikibot/version.py \
- pywikibot/weblib.py \
- pywikibot/xmlreader.py \
- scripts/__init__.py \
- scripts/add_text.py \
- scripts/basic.py \
- scripts/blockpageschecker.py \
- scripts/blockreview.py \
- scripts/capitalize_redirects.py \
- scripts/category.py \
- scripts/category_redirect.py \
- scripts/claimit.py \
- scripts/clean_sandbox.py \
- scripts/commonscat.py \
- scripts/coordinate_import.py \
- scripts/cosmetic_changes.py \
- scripts/create_categories.py \
- scripts/data_ingestion.py \
- scripts/delete.py \
- scripts/disambredir.py \
- scripts/djvutext.py \
- scripts/editarticle.py \
- scripts/fixing_redirects.py \
- scripts/flickrripper.py \
- scripts/harvest_template.py \
- scripts/illustrate_wikidata.py \
- scripts/image.py \
- scripts/imagerecat.py \
- scripts/imageuncat.py \
- scripts/listpages.py \
- scripts/login.py \
- scripts/maintenance/__init__.py \
- scripts/maintenance/make_i18n_dict.py \
- scripts/misspelling.py \
- scripts/newitem.py \
- scripts/noreferences.py \
- scripts/pagefromfile.py \
- scripts/patrol.py \
- scripts/piper.py \
- scripts/protect.py \
- scripts/replace.py \
- scripts/selflink.py \
- scripts/shell.py \
- scripts/spamremove.py \
- scripts/states_redirect.py \
- scripts/template.py \
- scripts/touch.py \
- scripts/transferbot.py \
- scripts/unlink.py \
- scripts/unusedfiles.py \
- scripts/upload.py \
- scripts/version.py \
- scripts/watchlist.py \
- setup.py \
- tests/__init__.py \
- tests/aspects.py \
- tests/utils.py \
- tests/api_tests.py \
- tests/archivebot_tests.py \
- tests/basepage_tests.py \
- tests/bot_tests.py \
- tests/cache_tests.py \
- tests/category_tests.py \
- tests/checkimages_tests.py \
- tests/cosmetic_changes_tests.py \
- tests/data/ \
- tests/data_ingestion_tests.py \
- tests/date_tests.py \
- tests/deletionbot_tests.py \
- tests/disambredir_tests.py \
- tests/djvu_tests.py \
- tests/dry_api_tests.py \
- tests/edit_tests.py \
- tests/edit_failure_tests.py \
- tests/exceptions_tests.py \
- tests/family_tests.py \
- tests/file_tests.py \
- tests/flow_edit_tests.py \
- tests/flow_tests.py \
- tests/http_tests.py \
- tests/i18n/ \
- tests/interwiki_graph_tests.py \
- tests/interwiki_link_tests.py \
- tests/l10n_tests.py \
- tests/link_tests.py \
- tests/logentry_tests.py \
- tests/mediawikiversion_tests.py \
- tests/namespace_tests.py \
- tests/oauth_tests.py \
- tests/page_tests.py \
- tests/paraminfo_tests.py \
- tests/proofreadpage_tests.py \
- tests/protectbot_tests.py \
- tests/pwb/ \
- tests/pwb_tests.py \
- tests/python_tests.py \
- tests/reflinks_tests.py \
- tests/replacebot_tests.py \
- tests/script_tests.py \
- tests/site_detect_tests.py \
- tests/tests_tests.py \
- tests/timestamp_tests.py \
- tests/timestripper_tests.py \
- tests/thread_tests.py \
- tests/tk_tests.py \
- tests/tools_chars_tests.py \
- tests/tools_ip_tests.py \
- tests/tools_tests.py \
- tests/ui_options_tests.py \
- tests/upload_tests.py \
- tests/uploadbot_tests.py \
- tests/weblib_tests.py \
- tests/weblinkchecker_tests.py \
- tests/wikibase_edit_tests.py \
- tests/wikidataquery_tests.py \
- tests/wikistats_tests.py \
- tests/xmlreader_tests.py
-
-deps = flake8>=2.2.5
- flake8-docstrings<0.2.2
+ flake8-putty
[testenv:nose]
commands =
@@ -249,6 +83,14 @@
max_line_length = 130
accept-encodings = utf-8
require-code = true
+putty-ignore =
+ generate_family_file.py : +D102
+ pywikibot/cosmetic_changes.py : +D102, D103
+
tests/deprecation_tests.py,tests/pagegenerators_tests.py,tests/site_tests.py,tests/textlib_tests.py,tests/wikibase_tests.py
: +D102
+ tests/ui_tests.py : +D102, D103
+
scripts/archivebot.py,scripts/casechecker.py,scripts/commons_link.py,scripts/cfd.py,scripts/featured.py,scripts/imagecopy.py,scripts/imagecopy_self.py,scripts/interwiki.py,scripts/replicate_wiki.py,scripts/solve_disambiguation.py,scripts/maintenance/compat2core.py
: +D102, D103
+
scripts/checkimages.py,scripts/freebasemappingupload.py,scripts/imagetransfer.py,scripts/lonelypages.py,scripts/movepages.py,scripts/nowcommons.py,scripts/redirect.py,scripts/isbn.py,scripts/reflinks.py,scripts/script_wui.py,scripts/templatecount.py,scripts/revertbot.py,scripts/weblinkchecker.py,scripts/welcome.py
: +D102
+
scripts/catall.py,scripts/imageharvest.py,scripts/makecat.py,scripts/maintenance/cache.py,scripts/maintenance/wikimedia_sites.py
: +D103
[pep8]
ignore = E241,E402,E731
--
To view, visit
https://gerrit.wikimedia.org/r/255910
To unsubscribe, visit
https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Id55b466b321357f2a6fc97ef5b93c0fee1f86d03
Gerrit-PatchSet: 2
Gerrit-Project: pywikibot/core
Gerrit-Branch: master
Gerrit-Owner: John Vandenberg <jayvdb(a)gmail.com>
Gerrit-Reviewer: Ladsgroup <ladsgroup(a)gmail.com>
Gerrit-Reviewer: Siebrand <siebrand(a)kitano.nl>
Gerrit-Reviewer: Xqt <info(a)gno.de>
Gerrit-Reviewer: jenkins-bot <>