!djTaTBQyWEPRQxrPTb:nixos.org

Nixpkgs Architecture Team

228 Members
https://github.com/nixpkgs-architecture, weekly public meetings on Wednesday 15:00-16:00 UTC at https://meet.jit.si/nixpkgs-architecture53 Servers

Load older messages


SenderMessageTime
8 Jan 2024
@philiptaron:matrix.orgPhilip 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
@Ericson2314:matrix.orgJohn Ericson infinisil: exciting stuff! I will try to finally make it. I left one small doc review on the thing 16:14:42
@Ericson2314:matrix.orgJohn 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:matrix.orginfinisil 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
@Ericson2314:matrix.orgJohn Ericsonoh ok :)16:20:01
@Ericson2314:matrix.orgJohn Ericsonoops!16:20:03
@philiptaron:matrix.orgPhilip 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
@philiptaron:matrix.orgPhilip 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:matrix.orginfinisil Philip Taron (UTC-8): PRs appreciated! 16:36:36
@infinisil:matrix.orginfinisilA normal comment < a comment with a suggestion < a PR16:37:42
@philiptaron:matrix.orgPhilip Taron (UTC-8)Perfect. I'll get you a normal comment first in case today is overcome by events. :-)16:38:35
@philiptaron:matrix.orgPhilip Taron (UTC-8)Delivered.16:39:14
@infinisil:matrix.orginfinisilThanks a lot!16:42:48
9 Jan 2024
@infinisil:matrix.orginfinisil @room: The next meeting will take place in a couple minutes - meeting link - meeting notes 13:57:10
@infinisil:matrix.orginfinisil tomberek: Robert Hensing (roberth): growpotkin ( Alex Ameen ): John Ericson: Ping? 14:04:56
@growpotkin:matrix.orgGrowpotkinI can't make it. I'm probably having a baby today!! 🎉 14:05:30
@infinisil:matrix.orginfinisil Growpotkin: Oh concrats! 14:05:53
@Ericson2314:matrix.orgJohn Ericsongah I am cursed with this law, was unable to sleep most the night and then finally fell asleep through this15:23:52
@Ericson2314:matrix.orgJohn Ericsonhttps://github.com/NixOS/nixpkgs/pull/278805 still unmerged?15:24:18
@infinisil:matrix.orginfinisil John Ericson: I'll merge it once I addressed the feedback :) 15:33:46
@infinisil:matrix.orginfinisil Philip Taron (UTC-8): Just addressed your feedback in https://github.com/NixOS/nixpkgs/pull/278805 :) 18:42:37
@infinisil:matrix.orginfinisilWasn't sure about one thing, but since it's not blocking anyways I'll merge soon!18:43:35
@philiptaron:matrix.orgPhilip 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:matrix.orginfinisilThought so, np :D22:06:54
@philiptaron:matrix.orgPhilip Taron (UTC-8)Want to run the rfc101 formatter on those test files?22:09:45
@philiptaron:matrix.orgPhilip Taron (UTC-8)

Copy-editing:

This should not be necessary, please move the package back and remove the manual callPackage.

Comma splice; remove the comma and replace it with either a period or semicolon.

22:11:19
@philiptaron:matrix.orgPhilip Taron (UTC-8) infinisil: reviewed, ✅ 22:29:13
@infinisil:matrix.orginfinisil
In reply to @philiptaron:matrix.org
Want to run the rfc101 formatter on those test files?
I'm not gonna bother since we're still changing it all the time :P
22:31:45
@philiptaron:matrix.orgPhilip Taron (UTC-8)The formatter, I hope, not the test files!22:32:04
@infinisil:matrix.orginfinisilHaha indeed, just the formatter22:32:25

Show newer messages


Back to Room ListRoom Version: 9