-
Notifications
You must be signed in to change notification settings - Fork 30
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
fix: make build parralel friendly #15
Conversation
This didn't include the parts from #10 that skips adding videos to irrelevant formats? Was that on purpose?
|
Ah, it's in #16 👍 |
I'm going to use squash merges from now so I try to split things up |
All good 👍 Looking forwards to the release! I did this write-up while trying to figure out how on earth to quickly put instruction videos in documentation: https://github.com/benjaoming/anti-pattern-sphinx-video-downloader Would be curious if you have considered various approaches and what you think is the best way forward for documentation projects in general? Are you going to put the documentation up on RTD? :) |
TBH the youtube solution in combination with sphinxcontrib-youtube works quite well for huge videos. Note that github LFS is not supported in RDT (last time I checked). Note that Drive videos behave the same as a youtube video and ca thus be wired the same way. On the other hand, for smaller one that you can afford to host in your docs (a 1 min demo of the product/tool) sphinxcontrib-video should do the trick. |
Git LFS is supported on RTD by extending the build process: https://docs.readthedocs.io/en/stable/build-customization.html#support-git-lfs-large-file-storage |
that's perfect, it seems I'm confusing with another service ;-) |
I'm not so clear in the description on https://github.com/benjaoming/anti-pattern-sphinx-video-downloader, but I suggest basically to have 2 repos active for documentation with video:
Note: The name of the video could be versioned such that old documentation can keep the original videos that they were published with. |
superceed #10