| 5 Jan 2024 |
infinisil | Yeah that continues working no problem | 01:25:43 |
infinisil | Profpatsch: CI will complain and tell you what to do if something isn't allowed :) | 01:26:23 |
profpatsch | but I guess it’s hard to transfer that into the new style then | 01:26:26 |
profpatsch | well, I’ll see | 01:26:42 |
infinisil | Future work, lots of future work! Small steps :) | 01:26:47 |
profpatsch | I don’t maintain that many packages anymore | 01:26:49 |
profpatsch | I hope at one point somebody does the work so much of the tranfer can be automated :) | 01:27:14 |
infinisil | Future work | 01:27:38 |
infinisil | Profpatsch: Automating the migration is also part of the RFC, that's gonna be next after enforcing it for new packages | 01:28:05 |
infinisil | Profpatsch: https://github.com/NixOS/nixpkgs/pull/211832 | 01:28:31 |
profpatsch | ah, nice | 01:30:02 |
| 9999years set a profile picture. | 21:01:26 |
| 9999years changed their profile picture. | 21:01:48 |
infinisil | @room Because it's a significant milestone, I'd like to schedule a meet for the final review and ideally merge of https://github.com/NixOS/nixpkgs/pull/275539 in ~2 weeks! For anybody wanting to join (but especially NAT members), please fill out https://crab.fit/making-pkgsbyname-mandatory-for-new-packages-908672 with your availability | 22:21:54 |
| @emilytrau:matrix.org left the room. | 22:32:09 |
| 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 |