Skip to content
This repository has been archived by the owner on Dec 16, 2022. It is now read-only.

Add nightly release schedule back #4123

Merged
merged 1 commit into from
Apr 23, 2020
Merged

Add nightly release schedule back #4123

merged 1 commit into from
Apr 23, 2020

Conversation

epwalsh
Copy link
Member

@epwalsh epwalsh commented Apr 22, 2020

This will run Sunday through Thursday night. It runs through the same workflow as a usual release except that the docs and docker images are not updated.

@epwalsh epwalsh requested a review from dirkgr April 22, 2020 21:50
push:
branches:
- master
release:
types: [published]
schedule:
- cron: '37 11 * * 0,1,2,3' # nightly at 11:37 UTC Sunday - Thursday
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just FYI, there was a preference before this on releasing 11 AM Seattle time: b4c7d76#diff-0aebd1990a83156b6c14a24f3a89f76eL6

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Was there a specific reason for 11 am? I made it earlier on purpose so that I would know if something went wrong first thing in the morning

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have no idea. I think @DeNeutoy put it that way, but idk the rationale 🤷‍♂️

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I was debugging it one day and I wanted it to run again that day, and it was about 10:45 😅 😅 😅 . Feel free to release whenever you like, it's not important.

@epwalsh epwalsh merged commit 5d58993 into allenai:master Apr 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants