| 9 Jul 2025 |
 Theuni | not sure why we'd go to the extra lengths of having them 600 and 640 ... | 09:34:27 |
 m1cr0man | Where is it inconsistent exactly? I don't remember all the permissions checks | 19:43:33 |
| m1cr0man | Sure, I'm sold :) | 19:47:44 |
|  @alina:catgirl.cloud changed their profile picture. | 21:01:28 |
| @alina:catgirl.cloud changed their display name from alina to alina arielle amelie🏳️⚧️🐾. | 21:02:13 |
| 10 Jul 2025 |
| Theuni | finally ... i got all tests working. one last cleanup regarding the lock handling, but then I should be ready for more eyes ... | 09:39:08 |
| Theuni | alright ... m1cr0man emily if you'd like to take a look https://github.com/NixOS/nixpkgs/pull/422076 is now ready. it's a lot more changes than I anticipated and I really tried my best to keep it down. | 12:43:41 |
| Theuni | hexa: if you have oppinions, then I'm all ear, too. | 12:43:59 |
| Theuni | Arian: and you taking another look is of course appreciated as well | 12:44:17 |
 emily | busy today and I'm sure m1cr0man will be more thorough than myself but I'll see if I can find time to take a quick look over the weekend | 14:16:11 |
| emily | frankly the module has grown so big that I find it hard to keep track of everything to review changes | 14:16:28 |
| emily | though I'd be very happy to review PRs that reduce the number of lines :D | 14:16:51 |
| Theuni | Understood. Overall it's a few more lines but I think the module itself is same length or shorter, but overall simpler. There's some places that could be DRY'd but at n=3 I'm still wary of early abstraction. Most new lines are in tests, I think. | 19:53:42 |
| 14 Jul 2025 |
| m1cr0man | I have been reviewing the change bit by bit for the last few days. Haven't had much time to sit down continuously. About 70% done | 07:34:45 |
| Theuni | thanks! i know it's a big one ... | 07:35:27 |
| Theuni | i'll be on vacation starting from thursday - no pressure, but don't expect a reply between thursday and august 5h. i'll pick this up afterwards if need be. | 07:35:55 |
 hexa | currently otherwise occupied with … mail. | 23:12:17 |
| hexa | * currently otherwise occupied with … mail stuff. | 23:12:20 |
| 15 Jul 2025 |
| m1cr0man | I'm on vacation until a similar time at EOM, so that works out :) I'll try and drop the review before my holiday | 00:00:08 |
| Theuni | 🙂 | 06:13:53 |
| 24 Jul 2025 |
|  blocklisted joined the room. | 10:10:53 |
| 28 Jul 2025 |
|  John joined the room. | 08:01:26 |
| 4 Aug 2025 |
| m1cr0man | Christian Theune: Just sent the review there. Sorry it took so long, I was on call the week before my holiday and was way too mentally exhausted to look at more code. | 14:43:09 |
| 5 Aug 2025 |
| Theuni | no worries. today's my first day back from vacation, so ... that fits perfectly 😉 | 06:15:24 |
| Theuni | (although that's not gonna be the first thing i'll touch today 😉 ) | 06:15:41 |
| 7 Aug 2025 |
|  Stéphan left the room. | 07:54:30 |
| 8 Aug 2025 |
|  0x4A6F joined the room. | 06:58:40 |
| Theuni | m1cr0man: great review, thanks! i'm wrapping things up on my side now (and found a few more minor details to polish). will poke here again in a bit once tests are clean on my side. | 13:24:58 |
| Theuni | alright. my PR is updated, there is some documentation issue that confused me (haven't touched docs before) that leona can likely help me fix next week. otherwise i've responded to all your comments and adapted the code here and there. | 14:19:49 |
| 9 Aug 2025 |
| m1cr0man | Just re-reviewed and approved it :) Thanks once again. It seems ofborg had an internal error when processing your last commit to the PR. you might have to trigger a force-push again.. or at least that's what I think will fix it. I've never seen that before. | 12:58:01 |
