about summary refs log tree commit diff
path: root/pkgs/test/nixpkgs-check-by-name/src/ratchet.rs
diff options
context:
space:
mode:
Diffstat (limited to 'pkgs/test/nixpkgs-check-by-name/src/ratchet.rs')
-rw-r--r--pkgs/test/nixpkgs-check-by-name/src/ratchet.rs4
1 files changed, 4 insertions, 0 deletions
diff --git a/pkgs/test/nixpkgs-check-by-name/src/ratchet.rs b/pkgs/test/nixpkgs-check-by-name/src/ratchet.rs
index dabc8f67316a7..10ecc01d3580c 100644
--- a/pkgs/test/nixpkgs-check-by-name/src/ratchet.rs
+++ b/pkgs/test/nixpkgs-check-by-name/src/ratchet.rs
@@ -69,6 +69,9 @@ pub enum RatchetState<Ratchet: ToNixpkgsProblem> {
     /// This is either because we already use the latest state, or because the ratchet isn't
     /// relevant.
     Tight,
+    /// This ratchet can't be applied.
+    /// State transitions from/to NonApplicable are always allowed
+    NonApplicable,
 }
 
 /// A trait that can convert an attribute-specific error context into a NixpkgsProblem
@@ -102,6 +105,7 @@ impl<Context: ToNixpkgsProblem> RatchetState<Context> {
             // Everything else is allowed, including:
             // - Loose -> Loose (grandfathering policy for a loose ratchet)
             // - -> Tight (always okay to keep or make the ratchet tight)
+            // - Anything involving NotApplicable, where we can't really make any good calls
             _ => Success(()),
         }
     }