Conversation
|
Nice idea 👍 just to clarify: this "only" removes the need to download the wheels each time, but pip still need to install them. |
precisely.
https://github.com/python-telegram-bot/python-telegram-bot/actions/runs/3825220803/jobs/6508026698 This takes 54s in the "install dependencies" tab, but it doesn't even include the one happening in pytest (after we test without dependencies). The precise time for that isn't available, but when I was looking at it live, it took about 30s or so |
|
Thanks for the link. Even if the advantage is not huge, I see no downsides of this change. I'll give the others a chance to chime in if they see the need and merge on thursday evening otherwise :) |
Attempts to speed up the test suite by caching GH actions dependencies. About ~1 mins is used every run just to install our dependencies. This should save that.
More info about how the cache is used (and also not used) can be found at https://github.com/actions/setup-python#caching-packages-dependencies
Edit: Confirmed the cache is working correctly by checking both https://github.com/python-telegram-bot/python-telegram-bot/actions/caches & the pytest logs in the workers below.