From cc910e2bc7421be139e29eaaaacc10f912dc1c06 Mon Sep 17 00:00:00 2001
From: Daniel Baker <daniel.n.baker@gmail.com>
Date: Tue, 17 Dec 2024 11:03:55 -0800
Subject: [PATCH] genemichaels: add comment about cargoTestFlags

Adds a rationale about why cargoTestFlags is not used.
---
 pkgs/by-name/ge/genemichaels/package.nix | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/pkgs/by-name/ge/genemichaels/package.nix b/pkgs/by-name/ge/genemichaels/package.nix
index f332cab62f6c..0decad7a3c83 100644
--- a/pkgs/by-name/ge/genemichaels/package.nix
+++ b/pkgs/by-name/ge/genemichaels/package.nix
@@ -18,6 +18,11 @@ rustPlatform.buildRustPackage rec {
   cargoHash = "sha256-FjggpBTzxj9AOJjUq5PmbuE/ImmTMpxN0se9uxRy4KQ=";
 
   cargoBuildFlags = [ "--package ${pname}" ];
+  # cargoTestFlags is not used because genemichaels is tightly coupled to the
+  # other crates in the workspace and by not setting it, we run all the tests.
+  # If a dependency crate is failing its tests, we want to know about it. For
+  # example, between versions 0.5.8 and 0.5.12, there was a failing test in one
+  # of the other workspace members that genemichaels depends on.
 
   meta = {
     description = "Even formats macros";