widgetti / solara

A Pure Python, React-style Framework for Scaling Your Jupyter and Web Apps
https://solara.dev
MIT License
1.62k stars 105 forks source link

CI: Run workflows once a week #347

Closed EwoutH closed 6 months ago

EwoutH commented 6 months ago

Adds a weekly scheduled run to all CI workflows which run on push or PR, which now run each Monday at 06:00 UTC.

A periodically scheduled run can detect errors and warnings appearing in changes in upstream dependencies or the build environment. By having a scheduled run they can be traced back easier to a dependency or environment change, then if they would pop up in a random PR.

maartenbreddels commented 6 months ago

Thank you! Note that we have bigger plans along the lines or https://github.com/widgetti/ipyaggrid/issues/44#issuecomment-1689710768 but I think this is indeed a good improvement. Although I think once a week is maybe not often enough. What about every day? Or twice a week?

EwoutH commented 6 months ago

Thanks for getting back! Since these workflows also run on every push to the main branch and on every PR, I think two additional runs a weeks should be more than enough. It’s more to have some recent reference run in periods of inactivity.

Monday and Thursday morning?

maartenbreddels commented 6 months ago

Could you add comments to the cron lines? I've never met someone in my life (except chatgpt) who knows by heart what it means. :)

EwoutH commented 6 months ago

Yes, will do!

I always use https://crontab.guru/ by the way, I find it really useful.

maartenbreddels commented 6 months ago

Dankjewel Ewout!

EwoutH commented 6 months ago

Graag gedaan!

EwoutH commented 6 months ago

The first scheduled runs where queued and performed as expected.

Screenshot_203

maartenbreddels commented 6 months ago

Love it ❤️ ! Thank you