about summary refs log tree commit diff
path: root/doc/README.md
diff options
context:
space:
mode:
authorValentin Gagarin <valentin.gagarin@tweag.io>2024-01-15 21:04:12 +0100
committerfricklerhandwerk <valentin@fricklerhandwerk.de>2024-01-15 21:04:12 +0100
commit0fe7b4c389fab957dc5d821341121c335c04497e (patch)
tree3ad528d6a7ea9905854aa424c2d24645ec3d6b91 /doc/README.md
parent4e977402c72241d693221e73ad064628e08a81c6 (diff)
doc: reword documentation conventions
Diffstat (limited to 'doc/README.md')
-rw-r--r--doc/README.md22
1 files changed, 13 insertions, 9 deletions
diff --git a/doc/README.md b/doc/README.md
index 43eb39c02ab18..b790b5f26e06a 100644
--- a/doc/README.md
+++ b/doc/README.md
@@ -159,24 +159,28 @@ In an effort to keep the Nixpkgs manual in a consistent style, please follow the
 In that case, please open an issue about the particular documentation convention and tag it with a "needs: documentation" label.
 
 - Put each sentence in its own line.
-  This makes reviewing documentation much easier, since GitHub's review system is based on lines.
+  This makes reviews and suggestions much easier, since GitHub's review system is based on lines.
+  It also helps identifying long sentences at a glance.
 
-- Use the admonitions syntax for any callouts and examples (see [section above](#admonitions)).
+- Use the [admonition syntax](#admonitions) for callouts and examples.
 
-- If you provide an example involving Nix code, make your example into a fully-working package (something that can be passed to `pkgs.callPackage`).
-  This will help others quickly test that the example works, and will also make it easier if we start automatically testing all example code to make sure it works.
-  For example, instead of providing something like:
+- Provide at least one example per function, and make examples self-contained.
+  This is easier to understand for beginners.
+  It also helps with testing that it actually works – especially once we introduce automation.
 
-  ```
+  Example code should be such that it can be passed to `pkgs.callPackage`.
+  Instead of something like:
+
+  ```nix
   pkgs.dockerTools.buildLayeredImage {
     name = "hello";
     contents = [ pkgs.hello ];
   }
   ```
 
-  Provide something like:
+  Write something like:
 
-  ```
+  ```nix
   { dockerTools, hello }:
   dockerTools.buildLayeredImage {
     name = "hello";
@@ -200,6 +204,6 @@ In that case, please open an issue about the particular documentation convention
 
   : Tag of the generated image.
 
-      _Default value:_ the output path's hash.
+    _Default value:_ the output path's hash.
 
   ```