| 8 Jan 2024 |
|  SomeoneSerge (back on matrix) changed their display name from SomeoneSerge (UTC+2) to SomeoneSerge (hash-versioned python modules when). | 04:50:15 |
 infinisil | I'd like to merge https://github.com/NixOS/nixpkgs/pull/278777 tomorrow in the NAT meeting. If anybody wants to review it more closely, please do so until then :) | 15:09:43 |
 @quantenzitrone:matrix.org | that pr is already merged | 15:43:02 |
| infinisil | quazi [ˈkvaːzi]: Ah whoops, messed up my copy-paste, thanks for mentioning, this is the one I meant: https://github.com/NixOS/nixpkgs/pull/278805 | 15:44:03 |
| infinisil | * I'd like to merge https://github.com/NixOS/nixpkgs/pull/278805 tomorrow in the NAT meeting. If anybody wants to review it more closely, please do so until then :) | 15:44:13 |
 Philip Taron (UTC-8) | infinisil: I'm taking a look. | 16:10:48 |
| Philip Taron (UTC-8) | I wrote some draft comments earlier in the week then got pulled away for work-and-family related stuff. None of them are blocking, just more refactoring for simplicity. | 16:14:11 |
 John Ericson | infinisil: exciting stuff! I will try to finally make it. I left one small doc review on the thing | 16:14:42 |
| John Ericson | * infinisil: exciting stuff! I will try to finally make it. I left one small doc review on the thing, but it basically looks good | 16:14:49 |
| infinisil | John Ericson: Thanks! Be aware that there's two PRs. One is just preparing with refactorings, that's the one I'd like to merge tomorrow (https://github.com/NixOS/nixpkgs/pull/278805). The one you reviewed is the one I'd only like to merge in the merge party in ~1-2 weeks :) (https://github.com/NixOS/nixpkgs/pull/275539) | 16:16:37 |
| John Ericson | oh ok :) | 16:20:01 |
| John Ericson | oops! | 16:20:03 |
| Philip Taron (UTC-8) | infinisil: would you be open a couple small commits targeting tweag:by-name-enforce-preparation or would you like to maintain the social lock on this tool? | 16:21:18 |
| Philip Taron (UTC-8) | In particular: switching anyhow::Context::context to with_context so that it doesn't make the error strings in the non-error case | 16:21:56 |
| infinisil | Philip Taron (UTC-8): PRs appreciated! | 16:36:36 |
| infinisil | A normal comment < a comment with a suggestion < a PR | 16:37:42 |
| Philip Taron (UTC-8) | Perfect. I'll get you a normal comment first in case today is overcome by events. :-) | 16:38:35 |
| Philip Taron (UTC-8) | Delivered. | 16:39:14 |
| infinisil | Thanks a lot! | 16:42:48 |
| 9 Jan 2024 |
| infinisil | @room: The next meeting will take place in a couple minutes - meeting link - meeting notes | 13:57:10 |
| infinisil | tomberek: Robert Hensing (roberth): growpotkin ( Alex Ameen ): John Ericson: Ping? | 14:04:56 |
 Growpotkin | I can't make it. I'm probably having a baby today!! 🎉 | 14:05:30 |
| infinisil | Growpotkin: Oh concrats! | 14:05:53 |
| John Ericson | gah I am cursed with this law, was unable to sleep most the night and then finally fell asleep through this | 15:23:52 |
| John Ericson | https://github.com/NixOS/nixpkgs/pull/278805 still unmerged? | 15:24:18 |
| infinisil | John Ericson: I'll merge it once I addressed the feedback :) | 15:33:46 |
| infinisil | Philip Taron (UTC-8): Just addressed your feedback in https://github.com/NixOS/nixpkgs/pull/278805 :) | 18:42:37 |
| infinisil | Wasn't sure about one thing, but since it's not blocking anyways I'll merge soon! | 18:43:35 |
| Philip Taron (UTC-8) | I'm reviewing https://github.com/NixOS/nixpkgs/pull/275539, infinisil . I accidentally clicked the single comment button. | 22:06:04 |
| infinisil | Thought so, np :D | 22:06:54 |
