DOC: Fix PEP-8 issues in 10min.rst #23908
Conversation
Signed-off-by: Fabian Haase <[email protected]>
doc/source/10min.rst
Outdated
| import pandas as pd | ||
|
|
||
| .. ipython:: python | ||
| :suppress: |
There was a problem hiding this comment.
why not at the beginning and with numpy and pandas imports in the same suppress block? I think this is what we do in all the other pages, right?
Signed-off-by: Fabian Haase <[email protected]>
| @savefig frame_plot_basic.png | ||
| plt.figure(); df.plot(); plt.legend(loc='best') | ||
| plt.legend(loc='best') |
There was a problem hiding this comment.
I think we've been ending those with semicolon in other parts, right?
There was a problem hiding this comment.
It's basically mixed sometimes with sometimes without.
I'd prefer to put ; at the end to suppress the output, but you've asked to PR that in seperate PR's and fixing PEP-8 issues is to have no ; at the end of a line.
I'm waiting for #23847 in order to ignore E703 on a file-based basis.
Codecov Report
@@ Coverage Diff @@
## master #23908 +/- ##
=======================================
Coverage 92.29% 92.29%
=======================================
Files 161 161
Lines 51498 51498
=======================================
Hits 47530 47530
Misses 3968 3968
Continue to review full report at Codecov.
|
|
thanks @FHaase |
No description provided.