| 22 Nov 2024 |
 maralorn | But let me not bog you down with feature ideas. | 15:01:07 |
| maralorn | I am not opposed to what you are proposing. | 15:01:27 |
| maralorn | But I am not 100% sure what you wanna do. | 15:01:43 |
| maralorn | Generally I think config state and data state should be decoupled. So I don’t think adding it to NOMV1State makes sense. | 15:03:05 |
| maralorn | (btw. that is terribly misnomed because it is at this point what I expected NOMV2State to be. 😄 ) | 15:03:30 |
| maralorn | * (btw. that is terribly misnamed because it is at this point what I expected NOMV2State to be. 😄 ) | 15:03:42 |
 Picnoir | Me neither tbh haha. The plan I was talking about above does not work either. The printer is abstracted out in writeStateToScreen (via OutputFunc, I don't really have way to inject a new tmvar to stateToText through it >< | 15:03:51 |
| Picnoir | * I'm not sure what I'm doing either haha.
The plan I was talking about above does not work either. The printer is abstracted out in writeStateToScreen (via OutputFunc, I don't really have way to inject a new tmvar to stateToText through it ><
| 15:04:13 |
| maralorn | Yeah, I recently tried to work on this and I think the abstraction design is a bit terrible right now.^^ | 15:04:31 |
| Picnoir | :) | 15:04:42 |
| Picnoir | It's a bit intense for my limited brain. | 15:04:49 |
| Picnoir | But I'm going to figure out a way to do this. We'll make it pretty later. | 15:05:10 |
| maralorn | The problem is that annoying Typeclass to support two relatively different modes (json and parsing human-readable stuff). | 15:05:53 |
| Picnoir | Right | 15:06:22 |
| maralorn | I might have time to work on nom tomorrow so I can look into what we need to improve/what would be the easiest way to inject your change. | 15:06:41 |
| Picnoir | Ok. I'll try to get something working by tonight. | 15:08:21 |
| Picnoir | (working != pretty :D) | 15:08:34 |
|  Morgan (@numinit) joined the room. | 17:49:28 |
| Picnoir | Download IMG_3364.mp4 | 18:26:24 |
| Picnoir | Hehe, got it working without having to move things too much around in the end | 18:26:54 |
| Picnoir | https://github.com/maralorn/nix-output-monitor/pull/162 | 18:27:30 |
| Picnoir | I'd like to add a couple of things later this weekend. | 18:27:42 |
| Picnoir | * Hehe, got it working without having to move things too much around in the end. The diff is pretty small. | 18:28:13 |
| maralorn | awesome! | 19:26:32 |
| maralorn | Maybe we want to strip the /nix/store prefix? | 19:27:00 |
| maralorn | I will definitely review this on the weekend. | 19:27:33 |
| Picnoir | In reply to @maralorn:maralorn.de
Maybe we want to strip the /nix/store prefix? I'd like to keep it, my use case is to copy/paste this string to a nix build call set with a remote builder :/ | 20:29:48 |
| Picnoir | Meh, nevermind, I can append that in my wrapper. Let's change that. | 20:30:19 |
| Picnoir | I fixed the update part. It's no longer waiting for a new nix line to come to display the derivation. | 20:31:03 |
| Picnoir | I have to confess I don't know how to fix the memoization issue rn. I have to figure out how this memoization thing is working. The documentation is terse and is pointing to dead links, rough start :D | 20:31:58 |
