LennartKloppenburg

Results 13 comments of LennartKloppenburg

Found it by inspecting the tests. Thanks anyway!

Of course. I have no idea if this is 'the right way' to do it, but I can expose the library like this: ``` //library.js /* The key is to...

@tatiana I've updated the PR with some changes you've requested :) One lingering issue: When there's no virtual env handy and multiple tasks are executed concurrently, they will all attempt...

> @tatiana I've updated the PR with some changes you've requested :) > > One lingering issue: When there's no virtual env handy and multiple tasks are executed concurrently, they...

Thanks so much for your answer @jlaneve :) > I like the locking idea! I’ve seen it work in other places well. A couple considerations: > > * what happens...

> Hey @LennartKloppenburg and @jlaneve , I see you advanced in the conversation and possibilities - I really like the way this is heading. > > Following the caching suggestion...

@tatiana I want to test things a bit more thoroughly but I think these changes should involve the gist of what we discussed :)

> Please let us know once you're happy with it. When you have a chance, please also rebase. Awesome, will do :) I plan to work on it again later...

> Thanks for addressing all the feedback and working on this thoroughly, @LennartKloppenburg ! Just had a quick look, and we're close to getting this change merged. > > Please...

@tatiana I rebased as well, but find that some tests are now failing. I have no idea why as they seem to be the same as the one in `main`...