-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Auto Generate API Documentation #2360
Conversation
@colin99d Was trying out with pandas doc yesterday and as a reflection, there are a few things I want to change with the current doc format.
So the updated one would look something like this. common.behavioural_analysis.sentiment_statsTo obtain charts, make sure to add Getting underlying datacommon.behavioural_analysis.sentiment_stats(ticker: str) -> DictGet sentiment stats [Source: finnhub]
Getting chartscommon.behavioural_analysis.sentiment_stats(ticker: str, export: str = '', view=True)
|
@minhhoang1023 got your changes in, I really like this new format. Feel free to let me know if you want any other changes. |
Does this feature a proper introduction page like https://openbb-finance.github.io/OpenBBTerminal/. If not, this has to be included. @minhhoang1023 @colin99d |
@colin99d This looks great Colin! 🚀 Once the API PR is merged into main, let's merge this bad boy too. For this introduction page, @JerBouma and I discussed we can make it a different PR. We still need to add in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much wow. Good Code.
Nice one colin |
Description
Automatically created documentation. #2352 needs to be merged before this PR is merged.
Things to think about:
How has this been tested?
Checklist:
Others
pre-commit install
.pytest tests/...
.