h5p / h5p-cli

Command Line Interface
MIT License
63 stars 31 forks source link

add worker_threads optimization for setup #90

Open devland opened 4 months ago

devland commented 4 months ago

Worker thread implementation essentially halves the setup time for h5p libraries that have many dependencies like h5p-course-presentation and h5p-column.

To-do: 1 - When using the H5P_SSH_CLONE=1 env var and an ssh key with a password added to the ssh-agent, since the cloning now happens in a separate thread, the ssh-agent context is lost and the cloning is done via https instead of ssh. So you cannot push to origin from the cloned repos. This needs to be fixed. 2 - Handle "folder already exists" cloning error as not an error (the same library might be cloned again by another worker thread running in parallel) and let other errors stop the setup as was the case before.