You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/03/15 03:52:12 UTC
[GitHub] [spark] HyukjinKwon commented on a change in pull request #27912:
[SPARK-31155] Enable pydocstyle tests
HyukjinKwon commented on a change in pull request #27912: [SPARK-31155] Enable pydocstyle tests
URL: https://github.com/apache/spark/pull/27912#discussion_r392638919
##########
File path: dev/tox.ini
##########
@@ -18,4 +18,4 @@ ignore=E226,E241,E305,E402,E722,E731,E741,W503,W504
max-line-length=100
exclude=cloudpickle.py,heapq3.py,shared.py,python/docs/conf.py,work/*/*.py,python/.eggs/*,dist/*,.git/*
[pydocstyle]
-ignore=D100,D101,D102,D103,D104,D105,D106,D107,D200,D201,D202,D203,D204,D205,D206,D207,D208,D209,D210,D211,D212,D213,D214,D215,D300,D301,D302,D400,D401,D402,D403,D404,D405,D406,D407,D408,D409,D410,D411,D412,D413,D414
+ignore=D100,D101,D102,D103,D104,D105,D106,D107,D200,D201,D202,D203,D204,D205,D206,D207,D208,D209,D210,D211,D212,D213,D214,D215,D300,D301,D302,D400,D401,D402,D403,D404,D405,D406,D407,D408,D409,D410,D411,D412,D413,D414,D415,D416,D417
Review comment:
Also, I think we didn't mention that we comply [PEP-257](ttps://www.python.org/dev/peps/pep-0257/) yet anywhere. Of course, It's good to incrementally fix to comply PEP-257; however, it ignores too many rules currently. I wonder if we added this too early without a comprehensive investigation. There were several rules that looked counterintuitive to me, IIRC. cc @BryanCutler who, IIRC, also pointed this out.
BTW, many rules were disabled due to our legacy Epydoc, which I managed to remove at https://github.com/apache/spark/commit/fe75ff8bea3330a10aba1a61f3aba42e541195a8. So, several rules might be able to enable back now.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org