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
docs: Rework live-preview to use docker container #11940
docs: Rework live-preview to use docker container #11940
Conversation
Please set the appropriate release note label. |
This way, we can rely upon the dependencies provided by the container rather than forcing folks to install various dependencies (even if mainly managed by pipenv). Signed-off-by: Joe Stringer <joe@cilium.io>
2796799
to
ccf2a86
Compare
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.
Nice!
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.
another big win here is that we don't get inconsistent spell check results because this uses the dictionary in the docker image.
having said that, live reload doesn't seem to work on mac :( i don't consider that a blocker though.
This way, we can rely upon the dependencies provided by the container
rather than forcing folks to install various dependencies (even if
mainly managed by pipenv).