-
Notifications
You must be signed in to change notification settings - Fork 566
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: add release schedule workflow #3268
Conversation
|
size-limit report 📦
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I love that you added the last week's issue and the conductor! Super helpful.
Do you think it would be helpful to add another table with the release tracking PR and the integration tests PR? It wouldn't be saving a lot of time but might help maybe to see them all in one place? If it is not too much of a hassle.
I like the timeline section! Once we have some more issues, we might start seeing a pattern when we release things or common problems etc.
Regarding the checklist item "Prerelease available on npm", do we need to associate it with a specific RC? Because it might be changing everyday depending on if there are more PRs merged to main. We could maybe ask from release conductor to add the RC version when they marked this checkbox maybe? Not sure if that would be helpful for anything though.
I really love this overall and will be a huge help for the handover notes! I don't have anything else to add at this point. I think we can start using if everyone feels keen and see what works and what not?
Thanks heaps for pushing this PR 💖
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love this! I think this will be super helpful
@broccolinisoup thanks for the review!
Great idea! Added some quick links for both that match what we have over in raycast 👀
Great points with this, I ended up updating that checklist section with what I used over in: #3275 Let me know what you think of the changes, the prerelease step ended up being removed and now it has checks for integration tests, stable release, and merging in the integration tests |
@joshblack I love the updates! Thanks so much for this 🙏🏻 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
Add in a workflow that will create a recurring issue for tracking releases. The template for this issue is roughly the following and can be updated:
Currently, the job is set to use
workflow_dispatch
while testing. In the future, this job should run on a cron schedule every Monday at the beginning of the week.Would love any feedback on this approach to see what you all think! The intent here is to keep track of our release process and maintain notes week-to-week.