Noissue - Added flake8-docstrings (sitescripts)

tox.ini

Index: tox.ini
===================================================================
--- a/tox.ini
+++ b/tox.ini
@@ -12,17 +12,22 @@
 omit = */test/*
 
 [flake8]
+ignore = D1

[Sebastian Noack, 2017/10/09 01:56:17]

D1 is for missing docstrings. I think enforcing docstrings on every
class/function/module goes to far. Let's just make sure that existing docstrings
comply to PEP-257.

[Vasily Kuznetsov, 2017/10/09 20:39:47]

We should decide what the policy should be for new code. I would say that
everything that's exported from a module (content of __all__, or everything that
doesn't start with an underscore if there's no __all__) definitely should have a
docstring, and maybe all toplevel functions as well as all non-private methods
also should have docstrings. In any case, for codebases like sitescripts blanket
ignore of D1 makes sense at this point.

[Sebastian Noack, 2017/10/10 04:38:32]

For library projects, I completely agree. However, most of our modules and their
functions/classes are not meant to be imported by other projects, and it this
case I usually advise: Add a docstring if you have anything to add that helps to
understand the code, don't add a docstring if it doesn't add anything that isn't
obvious from a quick look at the code.

 per-file-ignores =
     sitescripts/content_blocker_lists/bin/generate_lists.py : A107,E501
     sitescripts/crashes/web/submitCrash.py : A302,E501,E711,F401,N802,N806
     sitescripts/crawler/bin/import_sites.py : A105,A206,E501
     sitescripts/crawler/web/crawler.py : A107,A108,A206,E501
-    sitescripts/extensions/bin/createNightlies.py : A102,A104,A107,A108,A302,E501,F841,N802,N806,N812
-    sitescripts/extensions/bin/updateUpdateManifests.py : A107,A206,E501,N802,N806
-    sitescripts/extensions/utils.py : A107,A206,A301,A302,E501,E711,F401,N802,N803,N805,N806
+    sitescripts/extensions/bin/createNightlies.py : A102,A104,A107,A108,A302,D200,D202,D205,D208,D400,D401,D403,E501,F841,N802,N806,N812
+    sitescripts/extensions/bin/updateUpdateManifests.py : A107,A206,D200,D202,D205,D400,D401,D403,E501,N802,N806
+    sitescripts/extensions/test/conftest.py : D200,D400
+    sitescripts/extensions/test/test_createNightlies.py : D400
+    sitescripts/extensions/test/test_updateManifests.py : D401
+    sitescripts/extensions/utils.py : A107,A206,A301,A302,D200,D202,D204,D205,D208,D400,D401,D403,E501,E711,F401,N802,N803,N805,N806
     sitescripts/extensions/web/adblockbrowserUpdates.py : A107,E501
     sitescripts/extensions/web/downloads.py : E501,F401
     sitescripts/formmail/web/formmail.py : A301,E501,E713,N802
+    sitescripts/formmail/test/test_formmail2.py : D210,D400,D401
     sitescripts/hg/bin/irchook.py : A104
     sitescripts/hg/test/update_issues.py : N802
     sitescripts/management/bin/generateHgAuth.py : A107,E501,E711
@@ -45,9 +50,9 @@
     sitescripts/reports/web/showUser.py : A301,A302,E501,E711,N802
     sitescripts/reports/web/submitReport.py : A301,A302,E501,F401,N802,N806
     sitescripts/reports/web/updateReport.py : A301,E501,E711,F401,N802,N806
-    sitescripts/stats/bin/logprocessor.py : A101,A102,A104,A107,A204,A206,A301,A302,E501,E711,F401,N806
+    sitescripts/stats/bin/logprocessor.py : A101,A102,A104,A107,A204,A206,A301,A302,D202,D205,D208,D400,D401,E501,E711,F401,N806
     sitescripts/stats/bin/pagegenerator.py : A102,A104,A107,A301,A302,E501,E711,F401,N806
-    sitescripts/stats/common.py : A107,A112,E501
+    sitescripts/stats/common.py : A107,A112,D200,D205,D208,D400,D401,E501
     sitescripts/stats/countrycodes.py : E501
     sitescripts/stats/test/common.py : A107,A112,A301,E501
     sitescripts/stats/test/logprocessor.py : A101,A107,A112,E501
@@ -61,13 +66,14 @@
     sitescripts/subscriptions/combineSubscriptions.py : A102,A104,A105,A107,A206,A301,A302,E501,E713
     sitescripts/subscriptions/knownIssuesParser.py : A107,A201,E501,E711,E713,N802,N806
     sitescripts/subscriptions/subscriptionParser.py : A102,A107,A206,A302,E501,E711,N802,N803,N805
+    sitescripts/subscriptions/test/test_updateMalwareDomainsList.py : D400,D401
     sitescripts/subscriptions/web/fallback.py : A107,A206,A301,E501,F401,N802,N806
     sitescripts/templateFilters.py : A107,A112,A206,E501,E711,F401,F841,N802,N803,N806
     sitescripts/testpages/web/sitekey_frame.py : A107
-    sitescripts/urlfixer/bin/forceDomains.py : A206,E501,N802
-    sitescripts/urlfixer/bin/topDomains.py : A206,E501,E711,F401,N802
+    sitescripts/urlfixer/bin/forceDomains.py : A206,D205,D400,E501,N802
+    sitescripts/urlfixer/bin/topDomains.py : A206,D205,D400,E501,E711,F401,N802
     sitescripts/urlfixer/web/submitData.py : A206,E501,E711,F401,F841,N802,N806
-    sitescripts/utils.py : A107,A301,E501,N801,N802,N806
+    sitescripts/utils.py : A107,A301,D200,D202,D205,D208,D400,D401,E501,N801,N802,N806
     sitescripts/web.py : A107,E501,N802
     multiplexer.py : A107,E501
 
@@ -84,6 +90,7 @@
     pysed
     flake8
     flake8-per-file-ignores
+    flake8-docstrings
     pep8-naming
     hg+https://hg.adblockplus.org/codingtools#egg=flake8-eyeo&subdirectory=flake8-eyeo