-
Notifications
You must be signed in to change notification settings - Fork 11
Automating Managing LTS branches #2
Comments
IIUC correctly, right now we just backport every PR that has suitable semver labels? Or do |
@joyeecheung currently I use branch diff to generate a list of commits with a command line
and then manually review every commit. The labels are all manual too 😅 |
@MylesBorins OK I think I get what you meant now, including certain labels instead of excluding? Yes right now that looks like too much manual pings. |
@joyeecheung yup. We could definitely do that to flag the commits. We would need a bot that could poll for labels, land the commits, and push upstream. TBQH at that point the landing of the commit isn't the biggest deal... it is more about the review of what should / shouldn't land. Although in thinking about it, if the process of landing the commit was abstracted from the review process it would likely be easier to onboard more reviewers, and potentially inspire folks to review their own PRs. We would likely still need to audit the backlog of un labelled issues... but perhaps it would be easier to inspire people to label if it were an automated process... |
So, I've used https://github.com/evanlucas/node-backport for the past few releases I've done, and it sure makes things easier. Warning: It's pretty raw at this point though. |
We've already done lots of related work in the github bot, trying to backport PRs and add different labels: nodejs/github-bot/scripts/attempt-backport.js. Initially it also added I assume it something similar you're suggesting @MylesBorins with the addition of actually pushing changes to git branches? |
How could we automate the process of creating the staging branches?
An idea to start:
perhaps we could use labels to mark issues that are approved for backporting and a bot could automate cherry-picking it to a staging branch. If it does not land cleanly it adds a label / pings a backporting team
edit:
I've pivoted this issue to be about the LTS branches rather than the release process, as they are slightly different problems
The text was updated successfully, but these errors were encountered: