Nix Documentation | 406 Members | |
| Discussion about documentation improvements around the Nix ecosystem | 81 Servers |
| Sender | Message | Time |
|---|---|---|
| 17 Mar 2024 | ||
In reply to @fricklerhandwerk:matrix.orgindeed, the wiki is useful but I also find it goes into too much detail with flag setting. most of the time, if you set up things correctly, it should "just work". maybe I can add a section here in the meantime | 01:00:37 | |
In reply to @fricklerhandwerk:matrix.orgthat is good to see! I was actually thinking about the writers that live under scripts.nix (confusingly named indeed). these let you write python and many other languages inline and are, afaict not in the manual at all | 01:02:20 | |
| there is an aged issue about it: https://github.com/NixOS/nixpkgs/issues/89759 | 01:03:11 | |
| there is documentation in the source, so maybe it's an issue of making them render in the manual? not sure | 01:06:41 | |
In reply to @rina/:matrix.orgYeah it probably just needs to be wired up | 03:16:31 | |
In reply to @rina/:matrix.orgIf you could give a concrete example i can tell you if its a wiring issue or would need to be maintained seperately anyways in the long term. I remembered that the writers because some of them are dynamically created by partial application | 15:44:08 | |
In reply to @rina/:matrix.org* If you could give a concrete example i can tell you if its a wiring issue or would need to be maintained seperately anyways in the long term. I remembered that the writers because some of them are dynamically created by partial application and callPackages (plural s) | 15:44:33 | |
| * If you could give a concrete example i can tell you if its a wiring issue or would need to be maintained seperately anyways. I remembered that the writers have some difficulties. Because some of them are dynamically created by partial application and mapped via callPackages (plural s). | 15:48:12 | |
| Depends which methods we use for wiring them up. | 15:54:02 | |
| I am also not sure when it was decided to move reference documentation out of the source it is related to. I think this is a mistake. https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/trivial-builders/default.nix We still have nixdoc to retrieve doc-comments and it was extended recently. So wiring should only be a question of adding more files to the index. | 16:00:33 | |
| Expanding it should be question of adding more include blocks like this: https://github.com/NixOS/nixpkgs/blob/178fc9602881340bd8c02c0f9b64c25aedc0eb53/doc/doc-support/lib-function-docs.nix#L32C1-L39C34 | 16:01:45 | |
| * I am also not sure when it was decided to move reference documentation out of the source it is related to. I think this was mistake. https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/trivial-builders/default.nix We still have nixdoc to retrieve doc-comments and it was extended recently. So wiring should only be a question of adding more files to the index. | 16:02:14 | |
| * I am also not sure when it was decided to move reference documentation out of the source it is related to. I think this was mistake. I just took a look at this file: https://github.com/NixOS/nixpkgs/blob/master/pkgs/build-support/trivial-builders/default.nix It appears all doc-comments where removed from the code. Regardless we still have nixdoc to retrieve doc-comments and it was extended recently. So wiring should only be a question of adding more files to the index. | 16:04:32 | |
| Johannes Kirschbauer @hsjobeki: See https://github.com/NixOS/nixpkgs/pull/294944, I wouldn't call it being moved out of the source. Rather it was dedulicated in one place | 17:04:42 | |
| I'd also favor having it in the source code, but not having duplicated docs is more important than that | 17:05:27 | |
In reply to @infinisil:matrix.orgI'd like moving those back into the source code until we have a working system that allows to track external documents, so we dont loose track of the reference documentation. Maybe i'll open a PR soonish. More importantly we should work on reference tracking for those that are moved already. | 18:39:15 | |