-
Notifications
You must be signed in to change notification settings - Fork 670
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
Enabling RTD supported format for docs #680
Conversation
rsts/make.bat
Outdated
@@ -0,0 +1,36 @@ | |||
@ECHO OFF |
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.
can we just delete this file?
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.
I guess we can, it is auto-generated
rsts/requirements.txt
Outdated
@@ -0,0 +1,105 @@ | |||
# |
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.
can you add a make target to generate this file?
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.
I did add it, what happened, - oops overrode it
rsts/conf.py
Outdated
html_logo = "flyte_lockup_on_dark.png" | ||
html_static_path = ['_static'] |
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.
html_static_path = ['_static'] | |
html_static_path = [] |
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.
if you don't do this you just get a warning every time you build.
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.
ok
@@ -14,4 +14,4 @@ Getting Started | |||
|
|||
examples | |||
create_first | |||
more_examples | |||
Learn Flytekit by example <https://flytecookbook.readthedocs.io/en/latest/> |
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.
does this link work?
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.
yup
* Update pod example for local run Signed-off-by: Alekhya Sai Punnamaraju <[email protected]> * Grammar changes Signed-off-by: Alekhya Sai Punnamaraju <[email protected]> * Apply suggestions from code review Co-authored-by: Samhita Alla <[email protected]> Co-authored-by: Samhita Alla <[email protected]>
No description provided.