Slackbot
10/09/2023, 4:05 PMJacob Gold
10/09/2023, 4:12 PMStephen Caldwell
10/09/2023, 4:14 PMJacob Gold
10/09/2023, 6:43 PMStephen Caldwell
10/10/2023, 5:44 PMStephen Caldwell
10/10/2023, 9:08 PMJacob Gold
10/10/2023, 9:45 PMJacob Gold
10/10/2023, 9:45 PMJacob Gold
10/10/2023, 9:56 PMStephen Caldwell
10/10/2023, 10:22 PMAre you comfortable giving the Graphite app permission to bypass branch protection rules?Yes.
Do you care about the PRs within the stack showing the correct diff on GitHub after they have been merged?I would label that “nice to have”
Do you care that the PR shows “merged” on GitHub (as opposed to “closed”)?Not really.
If the first few PRs in the stack pass CI, but there is a failure midway through, should they all be removed from the queue, or should the passing PRs be merged?TL;DR - I want the entire stack to be merged all at once or not at all. I feel like a description of how CI works in my system is in order here. When a commit is pushed to a branch, CI runs some checks like build and test, and they show up as GitHub checks on the PR. When a branch is merged to trunk, it runs those checks plus a more extensive (integration) test suite, and then (if that passes) it deploys to production. So I don’t really know how CI relates to the merge queue. There’s nothing really for each individual PR to “pass” upon merge, other than going through the whole deployment process, which I’m trying to avoid.
Does your CD trigger on each commit to trunk or each push to trunk? I.e., do multiple commits being added to the trunk branch simultaneously trigger multiple CD runs?Each push to trunk.
Stephen Caldwell
10/10/2023, 10:25 PMJacob Gold
10/10/2023, 10:32 PMSo I don’t really know how CI relates to the merge queue. There’s nothing really for each individual PR to “pass” upon merge, other than going through the whole deployment process, which I’m trying to avoid.the merge queue (currently) runs your individual PR CI again after rebasing to ensure that the checks pass on the version that will actually be merged
Jacob Gold
10/10/2023, 10:32 PMStephen Caldwell
10/10/2023, 10:36 PMJacob Gold
10/10/2023, 10:46 PMStephen Caldwell
10/10/2023, 11:40 PMStephen Caldwell
10/11/2023, 12:08 AMIf the first few PRs in the stack pass CI, but there is a failure midway through, should they all be removed from the queue, or should the passing PRs be merged?I would like them all to be removed from the queue