-
Notifications
You must be signed in to change notification settings - Fork 246
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
Modified docs_build.sh to support --port option #515
Conversation
docs_autobuild.sh
Outdated
@@ -1,4 +1,9 @@ | |||
#!/bin/bash | |||
|
|||
cd docs && make clean && cd .. | |||
poetry run sphinx-autobuild docs docs/_build/html || printf '\n\nError running docs_autobuild. \nMake sure you are using poetry >= 1.1.15. \nCheck out https://docs.robusta.dev/master/developer-guide/platform/docs-contributions.html\n' | |||
|
|||
if [ "$1" == "--port" ];then |
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.
@pavangudiwada instead of the if/else can we always run poetry run sphinx-autobuild $@ docs docs/_build/html
and such that ./docs_autobuild.sh
supports all sphinx-autobuild
flags and passes them in automatically no matter what?
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.
Well, yep, it works without it. 😅
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.
Made changes
docs_autobuild.sh
Outdated
@@ -1,4 +1,9 @@ | |||
#!/bin/bash | |||
|
|||
cd docs && make clean && cd .. | |||
poetry run sphinx-autobuild docs docs/_build/html || printf '\n\nError running docs_autobuild. \nMake sure you are using poetry >= 1.1.15. \nCheck out https://docs.robusta.dev/master/developer-guide/platform/docs-contributions.html\n' | |||
|
|||
if [ "$1" == "--port" ];then |
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.
Well, yep, it works without it. 😅
Fixes OSError: [Errno 98] Address already in use error, caused when the default sphinx-autobuild port (8000) is reserved.