From c7f9378f341cbe5382ccfee237807b04c48613f8 Mon Sep 17 00:00:00 2001 From: Peder Bergebakken Sundt Date: Tue, 13 Aug 2024 20:23:41 +0200 Subject: [PATCH] nixos/README.md: add guideline to link to nixos tests from packages --- nixos/README.md | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/nixos/README.md b/nixos/README.md index 5f751e10e20a..2261803f4e7d 100644 --- a/nixos/README.md +++ b/nixos/README.md @@ -80,6 +80,7 @@ Reviewing process: - Ensure that all file paths [fit the guidelines](../CONTRIBUTING.md#file-naming-and-organisation). - Ensure that the module tests, if any, are succeeding. +- Ensure that new module tests are added to the package `passthru.tests`. - Ensure that the introduced options are correct. - Type should be appropriate (string related types differs in their merging capabilities, `loaOf` and `string` types are deprecated). - Description, default and example should be provided. @@ -95,7 +96,8 @@ Sample template for a new module review is provided below. ##### Reviewed points - [ ] module path fits the guidelines -- [ ] module tests succeed on ARCHITECTURE +- [ ] module tests, if any, succeed on ARCHITECTURE +- [ ] module tests, if any, are added to package `passthru.tests` - [ ] options have appropriate types - [ ] options have default - [ ] options have example