| 4 Oct 2025 |
 dramforever | seriously? GC Warning: Repeated allocation of very large block (appr. size 292 KiB): | 15:02:08 |
 Grimmauld (any/all) | i pressed the "rerun failed" button | 15:02:57 |
| Grimmauld (any/all) | lets see if this helps | 15:03:03 |
| dramforever | yay | 15:06:30 |
| Grimmauld (any/all) | oh god it'll ping reviewers? | 15:07:02 |
| Grimmauld (any/all) | eh, worth the experiment, but this might be "fun" | 15:07:21 |
| dramforever | i think it gives up if it'll auto ping > 15 users or something | 15:07:38 |
| Grimmauld (any/all) | ah yes indeed | 15:07:48 |
| Grimmauld (any/all) | there it goes, CI green :) | 15:07:54 |
| Grimmauld (any/all) | now how the hell is anyone supposed to review this thing | 15:08:09 |
 Lun | yeah it gave up "Too many reviewers skipping review requests" | 15:08:24 |
| dramforever | it's not like anyone reviews the auto merges right | 15:09:13 |
| Grimmauld (any/all) | yes, but the automerge bot is also more trusted than some "random" PR | 15:09:46 |
| Lun | by trusting your diff of the conflict fix or by doing the merge locally and diffing the end result? | 15:10:01 |
 hexa | at which point I could have done the merge myself 😄 | 15:10:33 |
| Grimmauld (any/all) | like, not that i am suggesting you'd be trying anything funny, but this would be a hard-to-spot place to inject some malicious blob. | 15:11:09 |
| Lun | in order to do the merge you must first do the merge | 15:11:12 |
| Grimmauld (any/all) | yeah XD | 15:11:19 |
| hexa | the compromise is obviously to make dramforever a committer | 15:11:54 |
| Grimmauld (any/all) | Ah got it! | 15:12:33 |
| dramforever | maybe i should have made it without the merge commit | 15:12:33 |
| Grimmauld (any/all) | git show HEAD will display the diff | 15:12:41 |
| Grimmauld (any/all) | and the diff is as advertised | 15:12:54 |
| Grimmauld (any/all) | and the commits before were unchanged | 15:13:00 |
| dramforever | oh, yeah | 15:13:09 |
| dramforever | if you have it locally you can check the --cc diff | 15:13:19 |
| Lun | I verified locally that the merge result matches except fixing the conflict | 15:13:29 |
| dramforever | (which is the default for git show on merge commits) | 15:13:33 |
| Grimmauld (any/all) | that too XD | 15:14:11 |
| dramforever | why does github not have this feature | 15:14:20 |
