20 Sep 2024 |
Artturin | In reply to @rosscomputerguy:matrix.org Easier to manage, GH restricts the size of the diff. Unless I want to build a PR, I wouldn't want to review in the terminal. GH's diff viewer is pretty convenient. There's simply no way to review this change manually, the only way is to copy the commands in the commit msg and diffoscope, but first drop the formatting commit because I didn't post the command for it(I'll add it later) | 02:45:03 |
Artturin | Maybe there's performance issues when rebasing and getting conflicts from a large commit? | 02:47:07 |
Artturin | That would be a valid reason to split it | 02:47:26 |
Artturin | * That would be a valid reason to split the change in to a few commits in the same PR | 02:48:10 |
Tristan Ross | Maybe but still like to review manually | 02:52:18 |
emily | you're going to manually review >4,000 files? | 11:30:51 |
emily | I think spot checks are the most we can hope for here | 11:31:02 |
emily | the important thing is just it getting 0 rebuilds | 11:31:18 |
Artturin | In reply to @emilazy:matrix.org the important thing is just it getting 0 rebuilds https://github.com/NixOS/nixpkgs/pull/341407#issuecomment-2353173805 | 11:32:55 |
emily | right :) | 11:33:19 |
emily | I shouldn't have said 0 strictly | 11:33:31 |
emily | I more meant "if we can account for every single rebuild that gives us stronger confidence we're not changing semantics than manual review could" | 11:33:47 |
| aleksana (force me to bed after 18:00 UTC) joined the room. | 14:13:25 |
| cafkafk joined the room. | 14:29:50 |
Tristan Ross | In reply to @emilazy:matrix.org you're going to manually review >4,000 files? No, I want to review the stuff I am a codeowner/maintainer of. | 16:47:04 |
Artturin | @rosscomputerguy:matrix.org: try files -> file filter -> files owned by you | 17:39:49 |
Artturin | https://github.com/orgs/community/discussions/40474#discussioncomment-4411127 | 17:40:18 |
Tristan Ross | In reply to @artturin:matrix.org @rosscomputerguy:matrix.org: try files -> file filter -> files owned by you Oh sweet, didn't even knew that button existed. Thanks. | 17:40:53 |
Artturin | Me neither | 17:41:44 |
Tristan Ross | Download image.png | 17:41:56 |
Tristan Ross | Oh, this isn't good | 17:41:57 |
21 Sep 2024 |
| linj joined the room. | 05:20:52 |
23 Sep 2024 |
| connor (he/him) (UTC-7) changed their display name from connor (he/him) (UTC-5) to connor (he/him) (UTC-7). | 17:57:50 |
24 Sep 2024 |
emily | Artturin: needs another rebase | 18:58:50 |
emily | bit of a race against time | 18:59:22 |
emily | I will hit the button as soon as it merges cleanly and ofborg evals cleanly, ping me if necessary | 18:59:38 |
Artturin | I don't think it'll be possible to have ofborg evalled and no conflict | 19:00:35 |
Artturin | We should let it eval now | 19:00:45 |
Artturin | then rebase and merge immediately | 19:00:51 |
emily | ok, works for me | 19:02:04 |