training-material icon indicating copy to clipboard operation
training-material copied to clipboard

Build videos on push to main now that build is fast.

Open hexylena opened this issue 3 years ago • 2 comments

Originally we built these on cron due to the very very long build times. We now build these alongside the deploy step which means they'll show up a bit faster, because we have very fast build times so it's not a huge waste of time for 95% of PRs which don't include video changes. Additionally this adds a second enhancement to not bother running the rest of the steps if no videos would be built.

There's no good way to test it :(

hexylena avatar Oct 20 '21 08:10 hexylena

oh, theres a merge conflict

shiltemann avatar May 19 '22 09:05 shiltemann

I think this is done now, ready @shiltemann

hexylena avatar May 19 '22 10:05 hexylena

This probably works fine, but, meh.

hexylena avatar Mar 30 '23 08:03 hexylena