feat(tvix/eval): warn about empty `inherit`s r/5604

Change-Id: I82bec6fe2210bcb88c46fd2fdf3e26bd613d1c1f
Reviewed-on: https://cl.tvl.fyi/c/depot/+/7768
Reviewed-by: flokli <flokli@flokli.de>
Reviewed-by: raitobezarius <tvl@lahfa.xyz>
Tested-by: BuildkiteCI

1 files changed, 6 insertions, 0 deletions

diff --git a/tvix/eval/src/warnings.rs b/tvix/eval/src/warnings.rs
index 9fb454e8147c..9ebf182f975d 100644
--- a/tvix/eval/src/warnings.rs
+++ b/tvix/eval/src/warnings.rs
@@ -15,6 +15,7 @@ pub enum WarningKind {
     InvalidNixPath(String),
     UselessBoolOperation(&'static str),
     DeadCode,
+    EmptyInherit,
 
     /// Tvix internal warning for features triggered by users that are
     /// not actually implemented yet, but do not cause runtime failures.
@@ -95,6 +96,10 @@ impl EvalWarning {
                 format!("this code will never be executed")
             }
 
+            WarningKind::EmptyInherit => {
+                format!("this `inherit` statement is empty")
+            }
+
             WarningKind::NotImplemented(what) => {
                 format!("feature not yet implemented in tvix: {}", what)
             }
@@ -113,6 +118,7 @@ impl EvalWarning {
             WarningKind::InvalidNixPath(_) => "W006",
             WarningKind::UselessBoolOperation(_) => "W007",
             WarningKind::DeadCode => "W008",
+            WarningKind::EmptyInherit => "W009",
 
             WarningKind::NotImplemented(_) => "W999",
         }