Added docstrings on app\examples#11872
Conversation
|
Thanks @ArchaeotheriumSapienter just a few final suggestions concerning Sphinx link alias (please push new changes to this same branch) |
0a57f55 to
af84790
Compare
|
I'm done with the changes in this Pull Request, thank you so much! |
bryevdv
left a comment
There was a problem hiding this comment.
Last comments:
- one unused alias
- edits to use consistent verbiage (always good to be consistent this way, to help situate users)
|
FYI if you go the files tab, you can apply suggestions in a batch from the web UI (if you want to use that method) |
|
Thank you for the input! I'll keep those in mind next time :) |
af84790 to
5b4a0a7
Compare
examples/app/weather/main.py
Outdated
| with the distribution option. | ||
|
|
||
| .. note:: | ||
| This chart needs the Scipy and Pandas package to run. See |
There was a problem hiding this comment.
Sorry I guess I missed one last one, will be good to merge after this:
| This chart needs the Scipy and Pandas package to run. See | |
| This example needs the Scipy and Pandas package to run. See |
There was a problem hiding this comment.
No worries! Its also on my part since I was not consistent, I'll keep your suggestions in mind next time :)
5b4a0a7 to
d77b4fa
Compare
|
Thanks @ArchaeotheriumSapienter ! |
|
Thank you! |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Added docstrings on app\examples to tackle #11540 with additional note blocks in some of the files.