| 12 Jan 2024 |
 danielsidhion | In reply to @mcdonc:matrix.org
danielsidhion: it's not on the list, but i'm starting to apply your function argument defintion list stuff to a PR I already have open... https://github.com/NixOS/nixpkgs/pull/277534/files This is amazing, thanks a lot! No need to update your PR again, but I just opened https://github.com/NixOS/nixpkgs/pull/280520 to add a few more conventions that I discussed in the latest meeting. | 16:31:33 |
 Chris McDonough | great... i'll try to put that file in shape | 16:35:10 |
 infinisil | In reply to @infinisil:matrix.org
Investigating.. Figured it out, 2.19.1 doesn't have the deprecation warning. Looks like that was only added in 2.19.2 | 16:35:37 |
| infinisil | And actually, the cachix-install-action was just updated to use 2.19.2 today, so maybe the homepage will update just by forcing a retry | 16:36:27 |
| infinisil | Trying it here: https://github.com/NixOS/nixos-homepage/actions/runs/7504900929/job/20432973247 | 16:37:21 |
| danielsidhion | Should we update https://github.com/NixOS/nixos-homepage/blob/master/scripts/update.sh to use nix flake update instead of nix flake lock then? | 16:37:51 |
| infinisil | danielsidhion: I might do that yeah | 16:39:00 |
| infinisil | (If 2.19.1 actually supports that already) | 16:39:33 |
| danielsidhion | I think it does, if the docs I'm reading are right | 16:40:20 |
| infinisil | Yeah | 16:42:32 |
| infinisil | But it doesn't support updating multiple inputs in a single command :/ | 16:43:53 |
| Chris McDonough | danielsidhion: if i could ask in context, it looks like https://github.com/NixOS/nixpkgs/pull/280520 adds an "Inputs" title... should there be a corresponding "Outputs"? | 16:45:16 |
| Chris McDonough | Or I guess "Output" heh | 16:46:36 |
| danielsidhion | Add an "output/outputs" if it makes sense. The example I put there might be somewhat bad and need rework, because I had in mind a function that outputs a derivation, so "passthru outputs" makes more sense | 16:48:58 |
| danielsidhion | For lib-style functions it might make more sense to use "output/outputs" | 16:49:34 |
| Chris McDonough | ok, i think that makes sense | 16:49:59 |
| Chris McDonough | in that trivial builders file, we have sort of haskell-style "runCommand :: String -> AttrSet -> String -> Derivation" style of documenting inputs and outputs... i'm going to remove those and go with that | 16:51:23 |
| infinisil | danielsidhion: https://github.com/NixOS/nixos-homepage/pull/1199 | 16:52:46 |
| danielsidhion | If that style of documenting inputs/outputs doesn't work well for your use case, please help improve the conventions by either adding more specific cases or helping find a common abstraction that works for more cases | 16:53:22 |
| danielsidhion | We're kind of making those conventions as I find the opportunities while working on the manual | 16:53:38 |
| Chris McDonough | yep... i don't personally think the "runCommand :: String -> AttrSet -> String -> Derivation" style is very useful so i'll replace it with what we talked about and we'l see what happens :) | 16:54:31 |
| Chris McDonough | if CI completes, could we merge https://github.com/NixOS/nixpkgs/pull/277534 as-is so far? rationale: i don't want to make more commits to it that are unrelated to the writeText* functions, which is what its about right now | 17:24:55 |
| Chris McDonough | (i'd like to make more changes to it that are related to function argument styles) | 17:25:15 |
| Chris McDonough | (but that are not related to writeText* funcs) | 17:25:46 |
| danielsidhion | I'm reviewing the pr right now | 17:33:26 |
 @jade_:matrix.org | In reply to @asymmetric:matrix.dapp.org.uk
i think we would really benefit from this. it would make it easier for people to contribute across the different projects on the platform, create links when appropriate, etc etc there's a bunch of efforts of unclear relation involving writing new docs builders. we really need to write down what they are and probably make a tracking issue to try to not step on our own toes. | 18:20:29 |
| infinisil | danielsidhion: Manual is now being updated again! https://nixos.org/manual/nixpkgs/unstable/ | 19:40:43 |
| infinisil | (merged the PR and manually triggered an update) | 19:40:56 |
| danielsidhion | Thank you for fixing it! | 19:41:26 |
| danielsidhion | In reply to @jade_:matrix.org
there's a bunch of efforts of unclear relation involving writing new docs builders. we really need to write down what they are and probably make a tracking issue to try to not step on our own toes. I think that's important, added this to the agenda for next meeting so it's not forgotten to be at least discussed, hopefully the issue is created during the meeting though | 19:43:53 |
