-
-
Notifications
You must be signed in to change notification settings - Fork 247
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
Automatically create PRs updating development with the new release #6101
Comments
/bounty 80 |
💎 $80 bounty • Mudlet🔨 See instructions for compiling Mudlet Steps to solve:
Thank you for contributing to Mudlet/Mudlet! Add a bounty • Share on socials
|
💡 @ibishal submitted a pull request that claims the bounty. You can visit your bounty board to reward. |
/attempt #6101 Options |
💡 @svallory submitted a pull request that claims the bounty. You can visit your bounty board to reward. |
Hey, @vadi2! I was going to leave this comment when I pushed the PR but totally forgot, sorry! A couple of things need to be defined before I can complete this:
Also, I see there are other bounties for some other steps of the release process (which has a lot more steps). If you want, we can discuss the automation of the entire process. It may be easier (and cheaper) to have a single dev do it all instead of splitting it into multiple bounties pieces. |
Hey @svallory! Thanks for your interest and thought. It seems we have 2 competing attempts on this issue, so I will include @ibishal in this discussion as well. Your observation is correct that the release process is currently pretty complex and takes a lot of time. That's why there hasn't been a Mudlet release in a while and we want to automate it to lessen the burden. @vadi2 gathered all the steps in https://wiki.mudlet.org/w/Release_Checklist. Regarding your questions here:
I hope this answers the questions. Please feel free to ask for clarification or further details. And if I got something wrong, I'm sure @vadi2 will correct me 😅 |
Hey @keneanung! The thing is that there are two ways this could go since, at this time, the branch creation requires manual intervention. Path 1:
this gives more flexibility in branch naming as it only needs to start with Path 2
This saves the step of manually editing the files, but then restricts the branch naming. Also, in case of errors like an invalid version in the name or issues updating the files, there's no easy way to recover from the action and the developer will have to delete the branch and create a new one. My 2 cents: I believe the automations are too granular and without a view of the automation flow (which will not necessarily follow the sequence on the wiki) it's hard to optimize steps individually. That's why I proposed creating a bounty to plan the whole thing before breaking it apart |
That sounds interesting, what do you have in mind? The overarching idea of these bounties is to streamline the whole release workflow. |
I think the best way to go about this is to schedule a meeting of 1-2hs with the person/people who is/are most familiar with the process. I don't mind helping you guys mapping this process out (regardless of getting future bounties or not). You guys can find me on discord as |
@svallory thanks for the offer! I've booked something in for next Wednesday at 8pm CET. Let's meet on our Discord: https://discord.gg/kuYvMQ9 |
Brief summary of issue / Description of requested feature:
Automatically create PRs updating development with the new release, ie #6100
Steps to reproduce the issue / Reasons for adding feature:
Less release burden.
The text was updated successfully, but these errors were encountered: