Skip to content

Commit

Permalink
Fix flake8 to skip specific cases (#865)
Browse files Browse the repository at this point in the history
Related with databricks/koalas#847 (review) ,

Fixed flake8 to ignore "docs/build/html/reference/api/*.py"

in my local test, it works well, and all plots visible in documentation like below:

<img width="927" alt="스크린샷 2019-10-03 오후 2 19 36" src="https://user-images.githubusercontent.com/44108233/66101387-a0d58800-e5e9-11e9-813d-fc48ce117327.png">
  • Loading branch information
rising-star92 committed Oct 3, 2019
1 parent 464b56c commit 02e9946
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
3 changes: 3 additions & 0 deletions databricks/koalas/plot.py
Original file line number Diff line number Diff line change
Expand Up @@ -1266,6 +1266,9 @@ def pie(self, **kwds):
... index=['Mercury', 'Venus', 'Earth'])
>>> plot = df.mass.plot.pie(figsize=(5, 5))
.. plot::
:context: close-figs
>>> plot = df.mass.plot.pie(subplots=True, figsize=(6, 3))
"""
return self(kind='pie', **kwds)
Expand Down
1 change: 1 addition & 0 deletions dev/lint-python
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,7 @@ flake8 checks failed."

echo "starting $FLAKE8_BUILD test..."
FLAKE8_REPORT=$( ($FLAKE8_BUILD . --count --select=E901,E999,F821,F822,F823 \
--exclude="docs/build/html/reference/api/*.py" \
--max-line-length=100 --show-source --statistics) 2>&1)
FLAKE8_STATUS=$?

Expand Down

0 comments on commit 02e9946

Please sign in to comment.